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
Wrong values gotten in the Cash Flow Statement of AMD #4
Comments
Hi Daniel, yes it's a strange issue of the api. I checked the financials and the figure on the website is the correct one. Interesting that the problem is only for 2017, the other figures are the same on the website and api. |
Hi Ljupch0, Thanks for making this yfinance package available in r. get_financials <- function (ticker, report_type="annual") { a <- dplyr::full_join(BS, CF) In my case that causes the extra rows. The reason is that some of the accounts appear in more than one of the financial statements. the full join without the reference to "ticker" and "date" only causes the extra rows to appear in the resulting output data frame. So I get six rows instead of 4. financials <- get_financials("ABBV", report_type = "annual") When I run the code snippet above, r informs me about the following: I do understand that it is very difficult to cater to all situations, and you probably structured your code so that it is more flexible. Here is the code I used to make it work for my purpose:#Income Statement Balance Sheetdf_BS <- get_bs(ticker="ABBV", report_type="annual") Cash Flowdf_CF <- get_cf(ticker="ABBV", report_type="annual") Combine the three financial Statements into one data frameannual_report <- df_IS %>% I attached a couple of screenshots below. Again, thanks for your hard work, much appreciated. |
Good catch @Tor-Storli! Feel free to do a pull request to fix the joins as I will not have time to fix this anytime soon. |
Hello Ljupcho,
Just to report a weird situation when retrieving the Cash Flow Statement of the ticker
AMD
.The Operating Cash Flow (annual) for 2017 is not the same shown in the CF Statement in the Y! Finance web page.
VERSUS
I do not believe is any issue with the package, but an issue of the API.
In any case, I thought it was worth to share it.
Thanks for the package!
Daniel.
The text was updated successfully, but these errors were encountered: