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

Risk-Return Tradeoff Model Notebook #6710 #6739

Merged
merged 27 commits into from
Oct 22, 2024

Conversation

HmbleCreator
Copy link
Contributor

Pull Request Template for OpenBB Developers

  1. Title: Advanced Risk-Return Analysis and Portfolio Optimization

  2. Why? :

    • To provide a more comprehensive and robust framework for financial analysis using OpenBB.
    • To enhance the existing portfolio optimization capabilities with advanced risk metrics and scenario analysis.
    • To improve code organization, error handling, and configurability for better maintainability and user experience.
  3. What? :

    • Implemented a modular structure with separate functions for each major analysis step.
    • Added advanced features including efficient frontier calculation, VaR/CVaR analysis, and scenario testing.
    • Improved data fetching with retry mechanism and enhanced error handling and logging.
  4. Impact :

    • Expected to significantly improve the depth and accuracy of financial analysis capabilities in OpenBB.
    • Provides users with a more comprehensive toolset for portfolio management and risk assessment.
    • Impact score: 8 (Medium-high impact due to extensive enhancements, but building on existing functionality)
  5. Testing Done:

    • Tested with historical data for multiple asset classes over a 3-year period.
    • Validated results against established financial models and benchmarks.
    • Performed stress testing with various market scenarios to ensure robustness of the analysis.

This notebook demonstrates advanced techniques for risk-return analysis and portfolio optimization using OpenBB. We'll explore various asset classes, implement modern portfolio theory, and utilize OpenBB's extensive financial analysis capabilities. This comprehensive guide covers:

Data collection and preprocessing
Exploratory data analysis with visualizations
Risk and return calculations
Efficient frontier computation
Portfolio optimization techniques
Advanced risk metrics (VaR, CVaR)
Performance attribution
Scenario analysis and stress testing
By the end of this notebook, you'll have a deep understanding of how to use OpenBB for sophisticated financial analysis and portfolio management.
@CLAassistant
Copy link

CLAassistant commented Oct 5, 2024

CLA assistant check
All committers have signed the CLA.

@jmaslek
Copy link
Collaborator

jmaslek commented Oct 14, 2024

Hey! This is some good work. Can you update the file name?

@jmaslek
Copy link
Collaborator

jmaslek commented Oct 14, 2024

/award 300

Copy link

oss-gg bot commented Oct 14, 2024

Awarding HmbleCreator: 300 points 🕹️ Well done! Check out your new contribution on oss.gg/HmbleCreator

@HmbleCreator
Copy link
Contributor Author

Hey! This is some good work. Can you update the file name?

yeahh! but What name do you suggest?

@jmaslek
Copy link
Collaborator

jmaslek commented Oct 14, 2024

Hey! This is some good work. Can you update the file name?

yeahh! but What name do you suggest?

Something related to risk-reward tradeoff that is notCOMMUNITY_EXAMPLE_TEMPLATE (1).ipynb

@HmbleCreator
Copy link
Contributor Author

HmbleCreator commented Oct 15, 2024

Hey! This is some good work. Can you update the file name?

yeahh! but What name do you suggest?

Something related to risk-reward tradeoff that is notCOMMUNITY_EXAMPLE_TEMPLATE (1).ipynb

The task of updating the file name has been completed. You can now proceed to review the updated notebook. If you have any further suggestions or need additional changes, please let me know!

I’m new to open source contributions, so I appreciate your patience if I make any mistakes. I'm eager to learn quickly, so please guide me as I navigate this process.

@HmbleCreator HmbleCreator reopened this Oct 15, 2024
@HmbleCreator
Copy link
Contributor Author

I have a question, Mr. @jmaslek: Was my PR accepted? I checked the details and saw that the "[General Code Linting] failed 20 hours ago in 47 seconds." I'm eager to understand what happened and what the error means, as I reviewed my code twice and couldn't find the issues mentioned in the logs.

@jmaslek
Copy link
Collaborator

jmaslek commented Oct 17, 2024

I have a question, Mr. @jmaslek: Was my PR accepted? I checked the details and saw that the "[General Code Linting] failed 20 hours ago in 47 seconds." I'm eager to understand what happened and what the error means, as I reviewed my code twice and couldn't find the issues mentioned in the logs.

Hey there are spelling errors:

./examples/Risk-Return Analysis and Portfolio Management in OpenBB:648: myu ==> my
./examples/Risk-Return Analysis and Portfolio Management in OpenBB:648: tHna ==> than
./examples/Risk-Return Analysis and Portfolio Management in OpenBB:1089: fOf ==> for
./examples/Risk-Return Analysis and Portfolio Management in OpenBB:1089: fPr ==> for, far, fps
./examples/Risk-Return Analysis and Portfolio Management in OpenBB:1089: fOf ==> for
./examples/Risk-Return Analysis and Portfolio Management in OpenBB:1099: bU ==> by, be, but, bug, bun, bud, buy, bum

@HmbleCreator
Copy link
Contributor Author

HmbleCreator commented Oct 18, 2024

The line 648, 1089 and 1099 all contain the images generated by the EDA section and performance attribution sections in the code, the only possible spelling errors that can be found are in those images:

Screenshot 2024-10-18 142443
Screenshot 2024-10-18 142548

other than that I couldn't find any spelling errors in the whole code, I have gone through them several times.
So I changed those Image encodings to base64, I hope this time it will work.

…ynb to Risk-Return Analysis and Portfolio Management in OpenBB.ipynb
…ynb to Risk-Return Analysis and Portfolio Management in OpenBB.ipynb
Images Encoding have  been changes to base64 to prevent spelling errors while merging
Cell Output Images Encoding has been changed to base64 which may have caused spelling errors.
@piiq
Copy link
Contributor

piiq commented Oct 22, 2024

closes #6710

@piiq
Copy link
Contributor

piiq commented Oct 22, 2024

/award 300

Copy link

oss-gg bot commented Oct 22, 2024

Awarding HmbleCreator: 300 points 🕹️ Well done! Check out your new contribution on oss.gg/HmbleCreator

@jmaslek jmaslek added this pull request to the merge queue Oct 22, 2024
Merged via the queue into OpenBB-finance:develop with commit d225e42 Oct 22, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants