-
Notifications
You must be signed in to change notification settings - Fork 142
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
balance_sheet ignoring 'trailing=True' (while cash_flow and income_statement do take it into account). #150
Comments
The fix (I think) should be something like this: def balance_sheet(self, frequency='a'):
return self._financials('balance_sheet', frequency) |
Absolutely agree. Thanks ! |
Hi, I am interested to do this. Can I try this? |
Please clarify the question (was not sure who the question is aimed at, and what you want to try). If the question was for me, and you wanted to look at the trend of a specific 'aspect/attribute', you can do it already like the below which will create a dict for the specific attribute and later you can visualize as applicable.
For attributes that are not in the balance_sheet, it is even slightly easier, since you don't need the quarterly report and you can just add the 'trailing=True' as parameter for the annual report to get also the data for the most recent 12 months. The original thread here was just to raise the question whether 'trailing=True' should be reported as a warning, or instead be allowed, resulting with a row from the most recent quarter added (i.e., mrq instead of TTM, in the case mrq is not included in the last raw of the annual report). |
@kushalnl7 It's all yours! And, just to be clear, the fix is really just removing the |
btw, not sure if to open a different issue (not a real issue for me, but could be helpful to others): when doing It still stays on 2.2.15 In order to update to 2.3.0, I did the following: Maybe it would be good to edit the 'installation' section in |
Describe the bug
I assume this is a bug rather than a feature request.
It seems that while when adding 'trailing=True' for cash_flow and income_statement at the TTM row, for balance_sheet it pretty much seems to ignore it (and the alternative is to have 2 calls, the second one being balance_sheet with frequency='q' in order to calculate manually the TTM line.
Not sure if this is a bug (ignoring the 'trailing=True') or a feature request, but I think it would be good (if not too hard) to make it consistent with the other reports and 'save' the need for the manual calculation after issuing a second call for the quarterly data.
Desktop (please complete the following information):
I used Windws11, Python 3.8, yahooquery.version = 2.2.15
The text was updated successfully, but these errors were encountered: