-
Notifications
You must be signed in to change notification settings - Fork 472
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
tmdb_lookup release dates #3637
base: develop
Are you sure you want to change the base?
Conversation
@gazpachoking let me know your thoughts on the bullet points |
694589b
to
0e8416a
Compare
a967af2
to
c785f4b
Compare
a4b845b
to
5b6d746
Compare
b6738e7
to
868424d
Compare
68f4a16
to
648e94c
Compare
c384527
to
61c6cf4
Compare
8c7c3e9
to
64d388d
Compare
d08d44f
to
c991bf1
Compare
9afb85b
to
46f8aa2
Compare
for more information, see https://pre-commit.ci
@gazpachoking ping |
This PR is stale because it has been open 150 days with no activity. Remove stale label or comment or this will be closed in 60 days. |
Wow, sorry about my lack of response here. I'm thinking if we add this, we should store it directly in our db, rather than relying on a memory cache. |
This PR is stale because it has been open 150 days with no activity. Remove stale label or comment or this will be closed in 60 days. |
Initial WIP of adding release dates to the tmdb lookup plugin.
Few notes:
timed_lru_cache
for lru_cache with cache expiration tho might be better to cache to disk or db to be more memory friendly