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.

Sign up for GitHub

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

Jump to bottom

[Bug] Stocks/candle - Candle is messed up. #4221

Closed
deeleeramone opened this issue Feb 11, 2023 · 1 comment · Fixed by #4231
Closed

[Bug] Stocks/candle - Candle is messed up. #4221

deeleeramone opened this issue Feb 11, 2023 · 1 comment · Fixed by #4231
Labels
bug Fix bug

Comments

@deeleeramone
Copy link
Contributor

Recreate by checking out the develop branch and then:

/stocks/load spy/candle

Screenshot 2023-02-11 at 12 31 01 PM

The bug is not present on the main branch. Same command, but with git checkout main instead.

Screenshot 2023-02-11 at 12 33 29 PM
@deeleeramone deeleeramone added the bug Fix bug label Feb 11, 2023
@deeleeramone
Copy link
Contributor Author

yFinance can be fixed by including auto_adjust = True to the download function.

With auto_adjust = True
image

With auto_adjust = False

image

@deeleeramone deeleeramone mentioned this issue Feb 13, 2023
Merged
@deeleeramone deeleeramone linked a pull request Feb 13, 2023 that will close this issue
Fixes bad yFinance candles with auto_adjust=True #4231
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Fix bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes bad yFinance candles with auto_adjust=True OpenBB-finance/OpenBB
1 participant
@deeleeramone