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

Required memoise function #37

Open
signalpillar opened this issue Oct 2, 2013 · 6 comments
Open

Required memoise function #37

signalpillar opened this issue Oct 2, 2013 · 6 comments
Labels

Comments

@signalpillar
Copy link

I haven't found memoise function implementation. I suggest to have at least two implementations with weakref and regular dictionary.

@kachayev
Copy link
Owner

kachayev commented Oct 2, 2013

Does "I suggest" mean that you are not going to implement it as PR?

@signalpillar
Copy link
Author

Hm, I will try to make my version and then PR. Thanks

@kachayev
Copy link
Owner

kachayev commented Oct 3, 2013

Just one question, if you are going to start implementation, how will it differ from http://docs.python.org/3.4/library/functools.html#functools.lru_cache ?

@signalpillar
Copy link
Author

thank you for pointing me this function. it appears in 3.2 only, while we need something similar in 2.7. also I think there must be something built on top of weakrefs. Btw, the signature of function is very interesting, I even haven't thought about maxsize and typed

@kachayev
Copy link
Owner

kachayev commented Oct 3, 2013

Ok, looking forward!

@signalpillar
Copy link
Author

I have found that there is backport http://code.activestate.com/recipes/578078-py26-and-py30-backport-of-python-33s-lru-cache/ of 3.2 implementation. What do you think about it ?

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

No branches or pull requests

2 participants