Skip to content

Assignment 2 of the visualization module#103

Closed
vanya-geo wants to merge 1 commit intoUofT-DSI:mainfrom
vanya-geo:assignment-2
Closed

Assignment 2 of the visualization module#103
vanya-geo wants to merge 1 commit intoUofT-DSI:mainfrom
vanya-geo:assignment-2

Conversation

@vanya-geo
Copy link

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

Adding the completed markdown file for assignment 2.

What did you learn from the changes you have made?

I learned that all choices in a graphic need to be very deliberate. The difference between the good and bad graphic was very stark. The good graphic tells a complete picture. The bad graphic was hard to read and missed important parts of the story.

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

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

How were these changes tested?

N/A

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

References to the visualizations (also in the markdown file)
Good: https://www.economist.com/graphic-detail/2015/05/12/seeking-safety?fsrc=scn%2Ftw%2Fte%2Fbl%2Fed%2Fseeking_safety&%3Ffsrc%3Dscn%2F=tw%2Fdc
Bad: https://www.infocaptor.com/sending-and-receiving-money-across-the-world-remittances-between-countries

Checklist

  • [X ] I can confirm that my changes are working as intended

@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.

@tianyi21 tianyi21 closed this Jan 29, 2026
@tianyi21
Copy link
Contributor

Thank you for submitting your Viz A2. Please follow the instruction to create the PR from your A2 branch to your main branch. Please let us know if you any help. Thanks!

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

Comments