Skip to content

Depend on cachetools instead of implementing our own cache - #118

Closed
yuvipanda wants to merge 2 commits into
mainfrom
ttl-cache
Closed

Depend on cachetools instead of implementing our own cache#118
yuvipanda wants to merge 2 commits into
mainfrom
ttl-cache

Conversation

@yuvipanda

Copy link
Copy Markdown
Member

More robust (but pure TTL) cache set up, with same 128 maxsize. https://cachetools.readthedocs.io/en/stable/ has more options we can switch to if needed.

@yuvipanda
yuvipanda requested a review from jnywong July 22, 2026 21:58
@yuvipanda

Copy link
Copy Markdown
Member Author

I also wasn't sure exactly how the existing code worked, as I noticed the _ttl param wasn't being used.

More robust (but pure TTL) cache set up, with same 128 maxsize.
https://cachetools.readthedocs.io/en/stable/ has more options
we can switch to if needed.
@yuvipanda yuvipanda closed this Jul 22, 2026
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.

1 participant