-
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
Financial Environment Segmentation #81
Conversation
@rohitinu6 The task/ Project has been done. You could check it and review it ASAP |
@alo7lika Please link the issue using keywords. (fixes/closes..) You can find the detailed info about the same here: Thanks & regards. |
Fixes rohitinu6/Stock-Price-Prediction #79 |
@Mayureshd-18 kindly review it. Thank you! |
@alo7lika Please use keywords as they are mentioned in the docs. Then only the issue would be linked to the PR correctly. Thanks and regards |
Till then @rohitinu6 @jvedsaqib please review the technical details and the file structure. Thanks |
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.
@alo7lika
Why are you using the image?
Also change the Installation section which should contain how to clone this repo and other necessary things.
Images have been used to make the project interesting and enhance the look. But will update as you requested |
@alo7lika |
@jvedsaqib @Mayureshd-18 @rohitinu6 All the changes have been made. |
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!
Financial Environment Segmentation
Closes #79
Description of changes:
Issue Reference: