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

Fix TypeError in /stocks/quantitative_analysis/beta_model #2190

Merged
merged 4 commits into from
Jul 29, 2022
Merged

Fix TypeError in /stocks/quantitative_analysis/beta_model #2190

merged 4 commits into from
Jul 29, 2022

Conversation

DiGyt
Copy link
Contributor

@DiGyt DiGyt commented Jul 27, 2022 •
edited
Loading

Description

How has this been tested?

  • Manually checking that no errors are raised for:
from openbb_terminal import api as openbb

>>

python terminal.py
stocks
load AAPL
qa

$

@DidierRLopes DidierRLopes added the bug Fix bug label Jul 27, 2022
@DiGyt DiGyt mentioned this pull request Jul 29, 2022
Closed
@DidierRLopes DidierRLopes merged commit f09e887 into OpenBB-finance:main Jul 29, 2022
@DidierRLopes
Copy link
Collaborator

Nice one @DiGyt !!

Thank you, this was a minor thing but crashed the entire qa. Great finding!

Enjoy your weekend 🥳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DidierRLopes DidierRLopes DidierRLopes approved these changes

Assignees
No one assigned
Labels
bug Fix bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DiGyt @DidierRLopes