-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
[IMPROVE] Increase the speed behind loading stock data #3699
Comments
hey @DidierRLopes I found a potential solution: I see that the bottleneck is in the API call. By using this is backed up by a closed issue from the yfinance project - ranaroussi/yfinance#851 let me know what you think. I will create a PR if team is happy with it. |
Hey @victortangggg, I didn't know this! This is definitely an improvement! Yes, please. Feel free to go ahead with the fix :) |
replace yfinance api call with stats to cut loading time
thanks! PR is here - #3700 |
I believe that
additional_info_about_ticker
function is responsible by most of this lag. I personally don't use that quick overview and find it annoying that it takes me this long to load data because of it. I think for default it shouldn't.See quick inspection below,

The text was updated successfully, but these errors were encountered: