Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Standardize cache operations into one module #97

Open
jnspitale opened this issue Oct 31, 2023 · 0 comments
Open

Standardize cache operations into one module #97

jnspitale opened this issue Oct 31, 2023 · 0 comments

Comments

@jnspitale
Copy link
Collaborator

OOPS relies on caching to optimize performance, so code to implement this occurs throughout OOPS, several instances of which are identical. A cache module would consolidate this functionality into one place and make it easier to maintain and test.

The benefit of this consoldidation is illustrated the frames module where several classes use identical code to implement caching. In the upgrade from python2 to python3 (where the .keys() and .values() methods work differently), not all of those modules were updated and the unit tests did not catch the error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant