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

CGPA prediction model #265

Closed
wants to merge 1 commit into from
Closed

CGPA prediction model #265

wants to merge 1 commit into from

Conversation

armaf002
Copy link
Contributor

Uploaded a complete CGPA prediction model notebook

Uploaded a complete CGPA prediction model notebook
Copy link
Collaborator

@Sammybams Sammybams left a comment

Choose a reason for hiding this comment

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

Thank you very much @armaf002 for your contribution. However, we will not be able to merge your pull requests as it doesn't follow the guidelines already stated in the starter notebook. There are different grading systems for different schools, so it was wrong to remove the data point that had a grading system of 10.

Also, the prediction is based on the normalised CGPA. So the CGPA is divided by the grading system to make it a minimum of 0 and a max of 1. With that, the grading system by the person cannot be an affecting factor. It shouldn't also be a metric to train the model as is a measuring metric, not a feature. This also means that your visualization of the distribution of the CGPA was wrong because grades of students that have a grading system of 4 operated will have their CGPA capped at 4 and hence the visualization tells a false story for CGPAs above 4.0.

So in all visualizations and predictions for this project, the normalised CGPA is what is being worked with. Thank you.

@Sammybams Sammybams closed this Nov 1, 2023
@armaf002
Copy link
Contributor Author

armaf002 commented Nov 1, 2023

Oh alright, thank you

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