-
Notifications
You must be signed in to change notification settings - Fork 407
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
Remove yahoo finance #65
Comments
Would it be correct to say that all that needs rewriting is this function in That sounds straightforward; would you like me to put together a PR? |
@eigenfoo Correct. |
It would be great to have an alternative that goes back further in time. We can probably use the yahoo SPY returns for the past, and then forward-fill with the ones from google. |
There's the IEX exchange API if it helps, and it requires no registration: |
@jaycode Yes, that seems like a more stable source! |
There is also some minimal updates for Pyfolio module for this to work. See here: jaycode/pyfolio@7ef8d27 A working example can be seen here: https://github.com/jaycode/pyfolio/blob/master/pyfolio/examples/single_stock_example.ipynb |
Yahoo finance has been discontinued for good. Google is a fallback (albeit a bad one) but there's no reason to display a warning and confuse users: quantopian/pyfolio#478 (comment)
The text was updated successfully, but these errors were encountered: