Skip to content

assignment-2 done#1

Open
SainaAliabadi wants to merge 3 commits intomainfrom
assignment-2
Open

assignment-2 done#1
SainaAliabadi wants to merge 3 commits intomainfrom
assignment-2

Conversation

@SainaAliabadi
Copy link
Owner

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

Looked at two examples of good and bad visualization and explained each of their characteristics and how they can be improved.

What did you learn from the changes you have made?

Based on the type of data we're trying to depict, we need to choose which type is better for visualization.

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

No

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

No

How were these changes tested?

Based on the slides we learned in class.

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

The McGill paper and the websites that were shared to find the examples for visualizations.

Checklist

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

Copy link

@PatelVishakh PatelVishakh left a comment

Choose a reason for hiding this comment

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

Assignment 1:

Incomplete/Pending Resubmission, Required Changes:

⚠️ Link the exact visualization. The referred visualization cannot be found.

✅ Both Classifications are well reasoned.

✅ Good Reasoning for both classification.

✅ Good Discussion on how to further improve Both Visualizations.

✅ Good Reasoning for Suggestions on improvements.

Vishakh Patel [LS]

As a learning support individual requested, you can find screenshots for my Assignment 2.
@SainaAliabadi
Copy link
Owner Author

Please find 2 screenshots committed as requested.

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