-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
[FR] - add provider for yfinance-cache #6159
Comments
@ValueRaider, thanks for reaching out! We basically run a wrapper around the It looks like a parameter for "use_cache" could be added where it is available, and then redirect the Python function accordingly. A couple of questions for you:
|
No.
Unofficially yes, implemented for unit tests. I'd have to add safety checks in case new location has old/incompatible data.
Almost-entirely Pickled Python objects, each with a small metadata dict. Prices are Pickled Pandas Dataframes.
Not a clue, I work synchronously. If this is fast then do you really need async? Btw since creating this FR, new release adds financials caching. |
I hope the benefit of persistently caching data is obvious - speed for user, less load on provider.
I've been working on a persistent caching wrapper for
yfinance
-yfinance-cache
. Basic idea is be smart about what & when to fetch. Supports a subset ofyfinance
- price history, calendar, shares outstanding, info. Currently finishing off financials caching.Still needs some polish, but now might be a good time to start thinking about integrating into OpenBB. API intended as a drop-in replacement of
yfinance
so should be easy. @deeleeramone I see you mostly handle theyfinance
provider, this feature might interest you. I did experiment creating a provider, but then discovered providers hardcoded anyway.The text was updated successfully, but these errors were encountered: