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

Jiancong Shen Assignment 5 #98

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jackshen1998
Copy link

No description provided.

```

#Plot tree
```{r}

plot(score_ctree)
```

Please interpret the tree, which two behaviors do you think the teacher should most closely pay attemtion to?

Choose a reason for hiding this comment

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

Try to answer this question.

Assignment 5.Rmd Outdated
D2$advice <- ifelse(D2$score>=0.8,1,ifelse(D2$score<=0.5,3,2))
accuracy<-nrow(filter(D2,prediction==1))/nrow(D2)

#The predicted score is 65% accurate.

Choose a reason for hiding this comment

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

Elaborate on it, as in, specify whether the model overfits or whether it is generalizable. Also, calculate the error rate of the model predictions.

accuracy<-nrow(filter(D2,prediction==1))/nrow(D2)

#The predicted score is 65% accurate.
```

Choose a reason for hiding this comment

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

Overall, great job on the coding part of this assignment but you may want to work on the analysis.

@@ -61,7 +61,7 @@ plot(score_ctree)
```

Please interpret the tree, which two behaviors do you think the teacher should most closely pay attemtion to?

The teacher should pat attention to prior_prob_count and hints, since they are two deteermining factors in the graph.

Choose a reason for hiding this comment

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

Try to elaborate on it based on the information from the decision tree.

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.

2 participants