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.

Sign up for GitHub

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

Jump to bottom

Cache ticker data to reduce API load #505

Open
StephanAkkerman opened this issue Feb 20, 2024 · 2 comments
Open

Cache ticker data to reduce API load #505

StephanAkkerman opened this issue Feb 20, 2024 · 2 comments
Labels
Difficulty: Hard 😭 This issue is hard to solve, a lot of code needs to be changed Improvement 📈 Improvement Priority: Medium 🥈 Assign this label if this issue is used around once a day

Comments

@StephanAkkerman
Copy link
Owner

StephanAkkerman commented Feb 20, 2024 •
edited
Loading

Maybe cache them for x minutes, this will help hitting the coingecko and yahoo finance rate limit.

How do we handle tickers that can be both stocks and crypto?
@StephanAkkerman StephanAkkerman added Improvement 📈 Improvement Priority: Medium 🥈 Assign this label if this issue is used around once a day Difficulty: Hard 😭 This issue is hard to solve, a lot of code needs to be changed labels Feb 20, 2024
@StephanAkkerman
Copy link
Owner Author

We can use the @cache decorator, with a certain limit on it so it is refreshed after some duration.

@StephanAkkerman
Copy link
Owner Author

Also cache TV indicators

@StephanAkkerman StephanAkkerman mentioned this issue Aug 7, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Difficulty: Hard 😭 This issue is hard to solve, a lot of code needs to be changed Improvement 📈 Improvement Priority: Medium 🥈 Assign this label if this issue is used around once a day
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@StephanAkkerman