Repository navigation
docs: explain default price adjustment and Close vs Adj Close (fixes #860) - #2992
Conversation
|
One wording problem in the new note: "Pass Checked against Yahoo's chart endpoint for AAPL around the 4:1 split on 2020-08-31 (
The 28 Aug close is 124.81, not the ~499 the stock actually printed that day, so Suggested wording for the second sentence: "Pass |
|
@arhancanli good catch — you're right, and thanks for the AAPL split evidence. Fixed in the latest commit: the note now says |
|
Not in readme, that needs to be absolutely minimal. Move your prose into the Sphinx and readme can link to it. |
|
Done — moved the prose to a new Sphinx page |
|
Do a git-squash https://ranaroussi.github.io/yfinance/development/code.html |
fb1343c to
722dcc3
Compare
|
Done — squashed to a single commit. |
|
I've made it absolute minimum |
Fixes #860.
Per @ValueRaider's direction on the issue ("Better to resolve through documentation"), this adds a short "A note on price adjustment" subsection to the README, right after the Main components list:
Ticker.history()/yf.download()default toauto_adjust=True, soCloseis already the adjusted close and no separateAdj Closecolumn is returned in this modeauto_adjust=Falsegives raw (unadjusted) OHLC plus anAdj ClosecolumnDocs-only change, no code touched.