Skip to content

Conversation

@mf72270
Copy link

@mf72270 mf72270 commented Nov 1, 2025

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

suggesting proper labels, accessible colors, and clear context to fix a confusing heat map.

What did you learn from the changes you have made?

That context and accessibility are essential for any effective data visualization.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

Yes, changing the chart type entirely to a bar chart or data table

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

Yes, improving the design without knowing what the original data represented.

How were these changes tested?

By checking color accessibility and ensuring the message was clear without the original context.

A reference to a related issue in your repository (if applicable)

This addresses Assignment 2's goal of analyzing and improving bad visualizations.

Checklist

  • I can confirm that my changes are working as intended

@github-actions
Copy link

github-actions bot commented Nov 1, 2025

Hello, thank you for your contribution. If you are a participant, please close this pull request and open it in your own forked repository instead of here. Please read the instructions on your onboarding Assignment Submission Guide more carefully. If you are not a participant, please give us up to 72 hours to review your PR. Alternatively, you can reach out to us directly to expedite the review process.

Copy link
Collaborator

@kunzhi-yu kunzhi-yu left a comment

Choose a reason for hiding this comment

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

Hi, can you please show me where your reference to the good visualization is?

@github-actions
Copy link

Hello, thank you for your contribution. If you are a participant, please close this pull request and open it in your own forked repository instead of here. Please read the instructions on your onboarding Assignment Submission Guide more carefully. If you are not a participant, please give us up to 72 hours to review your PR. Alternatively, you can reach out to us directly to expedite the review process.

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