UBC-MDS / DSCI532-peer-review

0 stars 5 forks source link

worldcovid-dashpython - Group10 #8

Open Kingslin0810 opened 2 years ago

Kingslin0810 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

Name of the app: group10-worldcovid-dashpython Link to the repo in milestone2: https://github.com/UBC-MDS/group10-worldcovid-dashpython/tree/v1.0.0 Link to the proposal.md file where it is the description of the persona and scenario: https://github.com/UBC-MDS/group10-worldcovid-dashpython/blob/main/docs/proposal.md


  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


manju-abhinandana commented 2 years ago

Reviewer: Group 9

Persona

  1. To analyze the trend over time of various COVID metrics, I believe an animated world map may not be the best suited visualization on the main page. Animated world map would be very effective during a presentation. But to draw insights the world map which shows data within a time range would have been more suitable. At certain periods there is no noticeable change in the color gradient and may not be very insightful.
  2. In the study since we are comparing Canada's approach to the pandemic with the other countries, cases per million, deaths per million would be a more useful metric than the number of cases or number of deaths.
  3. The plot names Current ICU Hospitalizations, Current Hospitalizations and New Vaccinations in the 3rd tab could be misleading as we are looking a trend over time. It could be replaced with 'Daily'.

Frontend / layout

Backend

All the widgets are working fine. The data fields are included in the tooltip for each plot and I found it very useful when looking at a data point and world map.

Documentation

Indicate something you would like to include in your own app after reviewing this one I would like to include different metrics in the dropdown for the user to select and visualize. I would like to include tabs/nav bars for our dashboard. I believe it would give a nice flow to the content.