Skip to content
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

Reports equity command failed to create ipynb notebook #4223

Closed
tan-yong-sheng opened this issue Feb 12, 2023 · 1 comment · Fixed by #4327
Closed

Reports equity command failed to create ipynb notebook #4223

tan-yong-sheng opened this issue Feb 12, 2023 · 1 comment · Fixed by #4327
Assignees
Labels
bug Fix bug crypto Crypto related notebookreports Notebook reports

Comments

@tan-yong-sheng
Copy link
Contributor

tan-yong-sheng commented Feb 12, 2023

Describe the bug
I expect when I type reports\equity, both ipynb and html equity report will be generated. It works for other reports such as crypto, economy, but only not working for equity report.

For your reference, here is the bug shown inside the ipynb file generated from the command \reports\equity:
Code_8IXBSbVXis

Referring to this image, I suspect that the "Long business summary" in the ipynb file is not longer extractable from yfinance anymore and so thus, it will generate a KeyError for that when we tried to run it on ipynb file:

For your info, I have also attached the ipynb file generated as the text file at the most bottom part of this bug report. Or you could refer here:

Text file (as .ipynb file format is not supported here): https://paste.ee/p/lKP3V
<Instructions: Copy the content of above file and then save this file as .ipynb file>

To Reproduce
Step 1: type reports
Step 2: type equity

Screenshots
WindowsTerminal_syhUbTxDyJ

Desktop (please complete the following information):

  • OS: Windows 11 Pro
  • Python version: 3.9.12

Installation method:

ipynb file created with error inside
Text file (as .ipynb file format is not supported here): https://paste.ee/p/lKP3V
<Instructions: Copy the content of above file and then save this file as .ipynb file>

@github-actions github-actions bot added bug Fix bug crypto Crypto related notebookreports Notebook reports labels Feb 12, 2023
@deeleeramone
Copy link
Contributor

Hi there, this is because of all of the yFinance issues. #4212 for example.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fix bug crypto Crypto related notebookreports Notebook reports
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants