Closed Fidasek009 closed 3 months ago
I was trying to follow the table layouts on the website.
Usually ok but sometimes Yahoo is dumb, we can deviate. Also Pandas works best with dates in index not columns (DatetimeIndex
etc)
Sure, I'll see what I can do 👌.
Can I get a review or some change proposals? Also the commit history looks awful 😅, I could create a new PR with just one commit if needed.
Looks good. Can you add some basic tests here: https://github.com/ranaroussi/yfinance/blob/459d5f69c20a757d8fa344e3cfe085063aeb4f6c/tests/test_ticker.py#L751
Git squash is the best way to tidy the commit history #1084
Can some one please merge this PR? would be good to get the estimates please. Mainly interested in revenue and earnings.
1 last thing I promise: add these new attributes to README
From my perspective it is ready. I also tried to find all issues and PRs related to this PR so you may have to go over those (you can find them in the initial comment).
ruff check yfinance/scrapers/analysis.py
I implemented most of the analysis page (https://finance.yahoo.com/quote/AAPL/analysis/). As the URL suggests, analysis is just a part of the
quoteSummary
API endpoint, which means all the data can be fetched from there. I saw some attempts at scraping the data (#1668 #1778), but that can be inefficient and slower.Preview
I was trying to follow the table layouts on the website. So far the outputs look like this (for AAPL):
Analyst Price Targets
Earnings Estimate
Revenue Estimate
Earnings History
EPS Trend
EPS Revisions
Growth Estimates
Related to:
1668
1778
1828
575
1280
1968
722
1856
678