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

allow ignoring some arguments for caching key #221

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fmigneault
Copy link

@fmigneault fmigneault commented Dec 7, 2021

This PR is not as complete or extensive as #202, but provides minimal implementation that fulfills the use case I'm facing.
It partially addresses #201.

This allows to decorate a function with some parameters to be ignored when creating the cache key that will match following calls.
It allows to group together some operations where a parameter is important for the calculation of the function, but is not strictly required to map the call to the cached result. This is particularly useful in situation where the parameter cannot be cached (not serializable).

@fmigneault
Copy link
Author

@bbangert
Hi. Is it possible to consider this PR?
Without this kind of functionality, I am not able to integrate beaker into my services.
I have used by patch in requirements, but this blocks me from using the official beaker for deploying to PyPI (disallows used of direct URL references).
Thanks

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

Successfully merging this pull request may close these issues.

None yet

1 participant