-
Notifications
You must be signed in to change notification settings - Fork 71
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
Peer Review Comments #46
Comments
Thanks for the feedback. We plan to extract data from Bloomberg data base, so right now we can't provide a usable link yet. We will work on the other suggestions! |
Please raise these comments on the repo for the project you're commenting on, not the class projects repo. Thanks! |
Hi professor Udell, I can submit my comments for the other group on their own repo, but I don't know why I cannot find the repo for this project. Whenever click the "issue", it brings me to here. |
The group should fix the settings on their repo; not sure why you'd be seeing that. @heyi19931225, any ideas? |
Good Things:
1.Intresting Topic
2.Proposal meets all the requirements
3. Organize the idea logically
Things for Improvement:
1.Justify why the problem is important
2.Add a data link
3.More details about how you will solve the problem (models, algorithms)
The text was updated successfully, but these errors were encountered: