firasm
commented
4 years ago - Below are some thoughts about the visualizations in this report and some suggestions for improvement, and notes when things are done well. You should implement these in milestone4
-
- [ ] THANK YOU (!!) for having a table at the beginning that has a human-readable version of the column/feature names. You could move this to the appendix if you find it's too long (but just remember to mention and link to it)
-
- [ ] It would really have helped if you had a caption and a figure/table number for each plot. That way you can easily reference it in your text, and it's easier for readers to find
- Figure 1:
- Plot is informative, bin choice is as good as it can be; effective for EDA purposes - show data imbalances where they exist
- Usually left-aligned titles are good/okay, but in this case, because you have two columns of figures, it's probably better to center the title
- Figure 2:
- Could you put figure 1 and 2 side-by-side? Separately they take up a lot of space
- Good clear headers and labels
- Not clear what y-axes show in each case.
- Oh I see, the y-axis units are in the header. Hmm... not sure how I feel about this. Interesting...
- Figure 3:
-
- [ ] Nice! This is a very powerful visualization that shows the reader exactly what you want to show
-
- [ ] Should make the text much larger here, quite hard to read
-
- [ ] Could add actual values on the plot itself if you round all values to 2 sig figs? will probably make it less messy and easier to read
-
- [ ] Consider setting diagonal values on your correlation map to be 0 or nan; those will really stand out otherwise.
-
- [ ] Ideally, you only want to show half the correlation map as the other half is redundant (see example here) - let me know if you figure out how to do this in Altair
-
-
- [ ] Figure 4:
-
- [ ] This plot is great!
-
- [ ] Figure 5:
-
- [ ] Excellent! Also a great plot, great job putting them side by side, makes it really clear
-
- [ ] Questionable colour choices though, can you try a different pallette ? The red is quite bright and the purple quite dim. Also, the text should probably be a different colour at the moment. Maybe white? See here how to change colours)
-
- [ ] Tables
-
- [ ] kudos for rounding your values to 3 digits - most people don't do this but doing it really improves readability. If it was me, I'd go to 2 sig figs but it's up to you
-
- [ ] Figure 6:
- [ ] Good putting these side-by-side. Did you see what it looked like if you put it on the same plot with the lines a different colour? Would probably be better for comparison
-
- [ ] Figure 7:
-
- [ ] Great! Stick to your convention of putting figures side-by-side for comparison. You also only need one legend for both
-
- [ ] Overall the report visualizations are pretty polished, but could use some work with sizing and formatting. Great work! Excellent work with your labels, titles, and plot choices! For every plot I was immediately able to understand what you were trying to convey.