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

Added export to stocks/qa/beta #2510

Merged
merged 8 commits into from
Sep 14, 2022
Merged

Conversation

hjoaquim
Copy link
Contributor

@hjoaquim hjoaquim commented Sep 6, 2022

Since beta and alpha are constants, I ended up doing this:

image

Is it correct/ the common practice?

@hjoaquim hjoaquim added the bug Fix bug label Sep 6, 2022
@deeleeramone
Copy link
Contributor

I'm not sure what the convention for this raw data would be; I was simply reporting that the export functionality didn't work. But, now that you have my attention, can this chart show the relationship to alpha over time?

Beta {ticker} as a +/- % value, relative to Alpha {SPY or user choice}?

@hjoaquim
Copy link
Contributor Author

@deeleeramone I would suggest for us to close the bug and open a new issue labeled "enhancement". Do you agree?

@colin99d colin99d marked this pull request as ready for review September 13, 2022 16:41
@jose-donato
Copy link
Contributor

@colin99d pingu

@hjoaquim
Copy link
Contributor Author

Following @jose-donato suggestion:
image

@DidierRLopes DidierRLopes merged commit d508c5e into OpenBB-finance:main Sep 14, 2022
@hjoaquim hjoaquim deleted the 2491 branch November 16, 2022 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fix bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Stocks/QA/beta: --export does not export
5 participants