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

finish! #109

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

finish! #109

wants to merge 1 commit into from

Conversation

Zifan96
Copy link

@Zifan96 Zifan96 commented Nov 19, 2019

No description provided.

#no action
T1[3]/sum(T1)

since everybody gets 1, and that means everybody should get no action. But our model says some students need monitor, so that means our model is not perfect!
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work. You can specify and report the accuracy rate of the model. To determine whether the model did a good job in prediction precision, we can compare it with a baseline model (prediction) which is unfortunately 100% here but should not be like this in the real-world dataset.

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.

3 participants