-
-
Notifications
You must be signed in to change notification settings - Fork 371
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
[IMPROVE] get_income_statement returns inaccurate results #94
Comments
Ah!! The issue here is that I automatically convert financial statements based on their currency. E.g. in this case I'd convert the statement to USD given that the historical data is also in USD (so any calculation that would use both actually works). This is also very relevant when you wish to compare companies. However, as the Income Statement contains the shares as well, which is not defined in a currency, this falsely alters these numbers. For now, disable this please by setting companies = Toolkit(
tickers=tickers,
api_key=FMP_KEY,
start_date="2019-12-21",
quarterly=False,
convert_currency=False,
# historical_source='YahooFinance',
)
print(companies.ratios.get_free_cash_flow_yield()) Thanks for bringing this up, I'll fix this issue this week. |
Thank you, I tried this, the income statement now matches the FMP endpoint output:
however
|
Hi! This is correct given that the stock price is in USD (which is used to calculate the market cap) and the financial statement is KWD. Please wait for the fix. |
Ah I see, thank you very much! :) |
Alright, I have adjusted the income statement so that the weighted shares are not adjusted by the currency. What is good to note in this case is that it is better to find the ticker in which both financial statements and historical data line up. For example, "Hyundai Motor Company" reports their financial statements in KRW however the ticker you have chosen is the ticker as shown on the American exchange. This means that the historical market data is depicted in USD. I automatically correct for this by finding the exchange rates and adjust the financial statement (so that the financial statement is the same as the historical data's currency) however this does lead to some discrepancies. Therefore, the more accurate result would be taking the ticker from the Korean exhange ("005380.KS", see https://sg.finance.yahoo.com/quote/005380.KS) which has both the financial statements as the historical data listed in KRW. I did notice that the weighted shares is different for both (which is why the numbers don't align as well as you would expect) in which I've reached out to FinancialModelingPrep to explain the difference. I'll let you know once they reply. In any case, do these numbers align better with your expections? Note that the market cap is based on the Weighted Average Shares. |
This issue has been resolved in v1.7.4, see: https://github.com/JerBouma/FinanceToolkit/releases/tag/v1.7.4 |
Hi! I have been validating get_free_cash_flow_yield ratio and can see that for some tickers it is not accurate.
Example:
If i try to get ratio for
HYMTF
in the following way:the result is:
but it cannot be larger than 1, right?
looking at the income statements, the
Weighted Average Shares
seems to be incorrect:while directly querying FMP endpoint
https://financialmodelingprep.com/api/v3/income-statement/HYMTF
gives more reasonable results at least for share count:do you have any ideas what's the issue here?
Thanks!
The text was updated successfully, but these errors were encountered: