-
-
Notifications
You must be signed in to change notification settings - Fork 25
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
Adj Close
in yfinance
has been removed
#563
Comments
I've had the same issue. My code was working fine last week, and it won't run this week as "Adj Close" is no longer accessible through the yf.download() method. Really annoying! Using unadjusted closing prices is completely useless - I will no longer use yfinance package in the future... |
Same. |
Should be fixed with #564! |
It is not fixed. Any changes to an API such as yfinance should be back compatible to ensure that people's code is not broken subsequent to changes. If you're planning on implementing breaking code, you need to have a user warning months in advance - otherwise users will just walk away and use a different API. If you go on Google Colab, the code below used to work and doesn't work anymore. !pip install yfinance Historically, if you were going to download one data series with yfinance, it would be 'Adj_Close'. If 'Close' has become 'Adj_Close' in the YF database, it's a very serious issue for everyone using the API. When I go on the web site, both the 'Close' and 'Adj_Close' columns are still shown in manually downloaded data - this implies that both columns are still in the underlying yahoo finance data. |
Thanks everyone for your comments - We definitely want to use |
This code https://github.com/ranaroussi/yfinance/blob/0713d9386769b168926d3959efd8310b56a33096/yfinance/utils.py#L445-L462 suggests that the |
I agree that it looks like it has been renamed, and that Close is actually now "Adj_Close". This is a really surprising change given that it breaks everyone's code. It would be interesting to understand the logical reasoning behind it given that the Yahoo Finance web site still shows both Close and Adj Close under the historical data tab. This change also breaks your code if you built your own custom Adj_Close using the old nominal Close (for example, an "after tax" custom series to measure the after tax performance of different dividend distributing stock portfolios). There are just so many reasons why this change is complete nonsense... |
Using 0.2.51 I added auto_adjust=TorF as example below:
Should get you back to having "Adj Close" |
In colab and locally, I am having trouble executing
prob_dist.md
:It might be related to this:
ranaroussi/yfinance#1333
The text was updated successfully, but these errors were encountered: