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
Improve color mapping for lines on charts #133
Comments
@sentientmachine started taking a look at this last night! |
Plan: Of the charts that show in the v2 student page, the color of the charts should be a smooth gradient between red and green to indicate an overall assessment of how good this rise is relative to the rise of the other students. That way if you see an increasing diagonal line that's green, you can know that this is the best of the group, if a decreasing diagonal line is green then that decline is the "smallest loser", otherwise if red it's the "worst of all users, who are also declining". |
🌔 🌝 😄 |
I've got the date of the leftmost position. I was able to get the indexes of the points to its left and right, and I was able to get the values of the points left and right. Next is to get the percentage the date is between the two dates, and then apply that same ratio to the y values to get the final leftmost y value. Then compare that to the rightmost value and apply a gradient between green and red for how much it goes up. |
@sentientmachine nice! 👍 |
I'm done with all the changes, I'm trying to do a git push to push my branch, but I get an authentication error: |
@sentientmachine nice! Yeah I think this is from trying to push to the main Student Insights repo. So I think what you'll need to do is:
I can take a look at this with you on Tuesday too if you want. Thanks for all your help! |
Across sparklines and details charts, particularly on the profile v2 page and the STAR charts, it'd be simpler for users if the colors for line charts had a consistent meaning. The STAR charts encode the change across the visible range with color (eg., red is regression, blue is growth), so this might be a good starting point for all charts.
The text was updated successfully, but these errors were encountered: