camkay / edld610_catd_finalproject

https://camkay.github.io/edld610_catd_finalproject/
MIT License
0 stars 4 forks source link

Peer Review #2

Open jrochel2 opened 5 years ago

jrochel2 commented 5 years ago

Cam, these are easily three of the coolest data visualizations that I have seen this year. But more importantly, I think they push beyond the "cool factor" and convey some very succinct stories that will stick with whoever gets the chance to view them. I am also impressed with your bravery, as quite a lot of zealotry comes along with the Star Wars fan base, but I do not think there is a fan alive that would not give these visualizations a solid stamp of approval. Your coding skills and attention to detail far surpass my own, so my feedback mainly will be limited to the actual visualizations and not the code. That said, I did learn quite a bit from your data manipulations and will be using the mutate(case_when()) in the future as it seems like a straight forward way to convert units into categories or vice versa. The palette choices across each visualization aligned really well with the Star Wars theme and were especially vivid on the black background.
In visualization #1, I thought the palette used was quite vivid and the pink text really worked well with the black background. You may have found the hidden secret on how to make bar graphs especially engaging, just add a light saber! Not to over analyze, but I wonder if you could go with a more "neutral" color for the male category light saber, like blue, given that red is associated with the dark side. This may be reading into things way too much, but just food for thought. I do think that the red vs green helps highlight the disproportionality of ages. For visualization #2, I do not think there is too much room for improvement here. The star fighter wings are a really neat way to show uncertainty. If there is a way to add a white shadow behind each starfighter, I think this could really help make each ship pop out. Finally, on visualization #3, I thought the turquoise text color was the right choice for representing the different planets, the blue conveys landmass. I think this graph is a bit harder to decipher than the previous ones, but that is more to do with the type of graph and not the way it is presented. One thought is that the circular rings for each AU are barely visible, so if those could be brightened then I think that would help clear up the distance scale. Overall great work, I wish I could go back to my data visualization presentation and use these graphs! @datalorax

datalorax commented 5 years ago

Nice feedback, Jon!