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

Swap Mean GPA for Median GPA #342

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

Conversation

AbhiramTadepalli
Copy link
Contributor

Overview

Median might be more representative of the distribution than mean, since median is resistant to outliers.

Other Notes

However, there is still a lot of discussion to be had because this will change how we present & how users perceive courses.

@AbhiramTadepalli AbhiramTadepalli linked an issue Jan 14, 2025 that may be closed by this pull request
Copy link

vercel bot commented Jan 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
utd-trends ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 14, 2025 3:53am

@AbhiramTadepalli
Copy link
Contributor Author

AbhiramTadepalli commented Jan 14, 2025

@TyHil what do you think about using median? For (ie) CS 2337, the median reported seems to represent the distribution a lot better imo.


##Some considerations

  • Median GPAs are boring (All medians of a B will be 3.33 always). I personally don't like this, but we don't have to show median gpa
  • What should be shown on hover?
  • Inside the card, can we show mean and median both? Or just mean? Or do away with mean completely?
  • What reasons do we have to not switch to median as the average statistic? I think across the board it has been boosting the average compared to mean...
  • Potentially combine median and mean, with weights? Too much to explain

@TyHil
Copy link
Member

TyHil commented Jan 14, 2025

Tbh my statistics knowledge is not up to par enough to know which to use. From basic googling it seems median is for skewed data which ours is likely to be so that seems like a reasonable switch.

Maybe keep average on hover ig.
We can add the median on the expanded card as a bar graph annotation instead of more text below the chart.

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.

[Feature Request] Use median GPA instead of Mean
2 participants