search

UBC-MDS / DSCI532-peer-review

0 stars 5 forks source link

safe_vancity_group_15 #10

Open arlincherian opened 2 years ago

arlincherian commented 2 years ago

To complete by the group

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

sukhleen999 commented 2 years ago

Group 14's review of Group 15: Safe Vancity

Hey team, great work on the dashboard! The app looks very impressive and is extremely helpful in analyzing crime statistics in Vancouver.

Persona:

Provide 3 pieces of constructive feedback that could help the app to be better suited to the scenario it was designed for:

  1. To make the map more interlinked with the other plots, it might be good to explore area selection in the map which will modify the remaining plots accordingly.
  2. Currently, the plots are neighbourhood specific. For future improvement, it might be nice for the app to have plots for 'All' neighbourhoods.
  3. The default view of the map after each selection of the crime type is quite zoomed out. A closer look into the area could be nice to review the spread of cases for the type of crime selected.

Provide 3 pieces of constructive feedback that could help the app to be better suited to the user(persona) it was designed for

  1. For someone with no technical knowledge of dashboards, it would be nice to include some information on how to operate the plots (something like a help button).
  2. It could be a good idea to have a summary table that ranks neighbourhoods by crime rate so that it is easy to compare them.
  3. A filter for the number of cases could be incorporated to limit the crime rate and filter the selection of neighbourhoods.

Frontend / layout

Is the text (plot labels / titles / subtitles) easy to read?

Are the plots selected appropriate for the data? Can you think of other alternatives or improvements for the plots regarding interactivity/reactivity or plot selection?

Was the dashboard organized in a way that was easy to read (columns / rows) or you would suggest a change in this sense? Would you prefer including tabs / navbars to improve the navigation or you think it is better to visualize everything in one screen by the nature of the content?

Are the plots or other images described properly for the persona described?

Backend

Are the options that you can select in the widgets working fine or have you found some cases where it is not working? If there are interactive plots: are the labels clear and easy to understand?

Documentation

Was the documentation easy to understand from the README.md how to use the app? How could the documentation be improved?

Was the app intuitive to use without reading the documentation? What information could be good to include?

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

arlincherian commented 2 years ago

Hello reviewers from Group 14!

Thank you so much for taking the time to provide us constructive feedback. Here are some ways we have implemented changes you suggested.

  1. Comment: To make the map more interlinked with the other plots, it might be good to explore area selection in the map which will modify the remaining plots accordingly.

  1. Comment: Currently, the plots are neighbourhood specific. For future improvement, it might be nice for the app to have plots for 'All' neighbourhoods.

  2. Comment: The default view of the map after each selection of the crime type is quite zoomed out. A closer look into the area could be nice to review the spread of cases for the type of crime selected.

    • The poorly map has the function to zoom in and out. We believe this functionality works and is sufficient as the user can zoom into specific streets/neighbourhoods etc. The scatter map is also centred around a specific latitude/long, therefore, increasing the zoom size too much means the map plot maybe zoomed into a specific location and the user will have to zoom out to see the density in specific neighbourhoods.
    • Commit Link: https://github.com/UBC-MDS/safe_vancity/commit/31b30c24dff6b0d1cc0fd1cdb630d82f22932509

  3. Comment: For someone with no technical knowledge of dashboards, it would be nice to include some information on how to operate the plots (something like a help button).The plots in the app are more or less self-explanatory. Although, it would be good to have a small section that documents the key takeaways from the app and how to use it efficiently.

  4. Just a suggestion - it would be nice if the horizontal bar chart would highlight the crime type selected in the filter on the left.