Peer Review

[CChCheChen]

Hi @Althrun-sun,

I can only review your app by cloning the code and test it on my local.

First of all, the file path to load the data is not correct and it works after I updated to 'pd.read_csv("../data/car_price.csv")'.

Once the app loaded, a few improvement can be implemented.

1. Filters should be placed in the section which user are easier to check, for example on top or on the left.
2. Filters could be more responsible to update the plot, without clicking on the apply filter button
3. The pie chart is not suitable to display the market share for each company, and this plot seems to no respond to the change in filters.

Overall, interesting app to play with!!!

[Suraporn]

Congratulations for the app!!

• It is a very nice-looking app with a proper theme and color.
• The documents (Readme, Proposal, Contributing, Code of Conduct, License) are well prepared.
• One thing I may suggest for the visualization is that it will be better if the pie_chart is limited with some number of car manufacturers since it is difficult to see when we plot all manufacturers.
• Also the line chart may not properly work yet. I chose Fuel and 918 Spyder and I did not see the line chart result.

[pengzh313]

It is a nice and meaningful visualization for people to explore the used vehicle market.

There are two things that might be considered for further improvement:

1. Add a overall title for the app to give users a clear idea about what the app is doing if they do not have chance to see your repo.
2. The pie chart seems not effective to distinguish so many car manufacturers by colours. Maybe a simple bar chart would be more effective if we want to see a ranking of market share of car manufacturers.