Issue #1 - Pie Chart - Does it make sense?

JulieBousquet / CDD_Step_Peer_Review_2018

Peer Review Folder 2018 - CDD Project

0 stars 0 forks source link

Open JulieBousquet opened 5 years ago

JulieBousquet commented 5 years ago

Do the code and graph make sense? In general, would you have done the same and do you have any suggestion to improve the graph?

escandethomas commented 5 years ago

I think the pie makes a lot of sense and is easy to read. A few extremly minor comments:

I would write the .53% as 0.53%
In the legend, some words don't have capital letters when they should (Storage, market,...) should be (Storage, Market,...)
Some slices have different sizes but I guess it is difficult to change.