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

Midterm peer review #119

Open
jk2332 opened this issue Nov 14, 2019 · 1 comment
Open

Midterm peer review #119

jk2332 opened this issue Nov 14, 2019 · 1 comment

Comments

@jk2332
Copy link

jk2332 commented Nov 14, 2019

First, plots in the report provides meaningful analysis into the data.
In the next section, the report mentions to fix overfitting by putting in more data. If distribution for other data remain relatively the same with the current data, adding in more data won't necessarily fix the underfitting issue unless the model is changed or more features are added. I think it would be great if other models tot be tried out in the future, if any, are mentioned.
The current model is mentioned to be underfit but in case that overfitting happens in the future, plans to resolve such issue could be mentioned as well.
In general, this report provides meaningful analysis into the data. I think it would be more informative if it incorporates more on future models and plans for data analysis.

@sijiafan
Copy link
Collaborator

please leave your peer review comment on the group project's repo, but not on this ProjectsFall2019 repo.

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

No branches or pull requests

2 participants