UBC-MDS / DSCI532-peer-review

0 stars 5 forks source link

NetViz - Group 3 #7

Open lynnwbl opened 2 years ago

lynnwbl commented 2 years ago

DSCI532-peer-review

  1. Open an issue with the name of your group and app Use this template:

To complete by the group

  1. Review the group that is one number ahead of your group's number. For example, if you are group 5, review group 6. If you are group 30, review group 1. Paste your answers in the issue of the group assigned.

To complete by the group reviewing:

Persona

Frontend / layout

Backend

Documentation

Indicate something you would like to include in your own app after reviewing this one


michelle-wms commented 2 years ago

Persona

Frontend / layout

Backend

Documentation

The documentation is fairly easy to understand. It’s very nice that the table of content was included in the README.md file so that users can quickly navigate through the documentation. And the link to the data set and app were both included, which is very handy. In the description section, the document clearly narrated how to use the dashboard step by step. Users can have a clear idea of what to expect from each visualization module. However, it would be better if more explanation could be provided about the movie rating. This part is not that intuitive. It’s really nice that the gif of app usage was included in the readme to show the function of the widget. I really like this part. The installation part included very detailed installation instructions to help users install and run the app locally, which is very nice and intuitive. Overall, the documentation was very clear and intuitive for users to use and follow. One thing I would recommend is that the motivation is better placed before the description of usage. As I was a little confused about the target users and the purpose of developing this app at the beginning. The app was intuitive to use without reading the documentation. But it's better to include a short description of each visualization for users to follow the information displayed in each plot. For example, in the plot named “World map”, I found it hard to understand what the app was showing. It could mean how many users in different countries were using Netflix as well without providing sufficient context information.

Overall, great job and very informative dashboard! 👯