-
Notifications
You must be signed in to change notification settings - Fork 83
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
Market Trend Classification Model #128
Conversation
@rohitinu6 I have completed the task. Please review it. |
@Mayureshd-18 kindly review the task |
@rohitinu6 @jvedsaqib Kindly add labels on PR and merge and Close the issue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Merging!
Market Trend Classification Model
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MastanSayyad review this PR
Closes: #84
Description of changes:
The current implementation of the Market Trend Classification Model provides a strong framework for identifying and classifying market regimes based on historical stock price data. However, it lacks functionality for:
Real-Time Market Adaptation: The model does not account for dynamic updates based on real-time market conditions, leading to a delay in identifying emerging trends.
Incorporation of Alternative Indicators: The existing model primarily uses historical price data and basic technical indicators. It does not include alternative data sources such as sentiment analysis from news and social media or macroeconomic indicators, which could significantly enhance model accuracy.
Visualization and Interpretability: There is no interactive dashboard or visualization tool for users to easily interpret the model's output. This limitation can hinder understanding and reduce the model's usability for non-technical users.
Issue Reference: