UBC-MDS / DSCI532-peer-review

0 stars 5 forks source link

Olympic_athletes_dashboard - Group 8 #17

Open cuthchow opened 2 years ago

cuthchow commented 2 years ago

DSCI532-peer-review

PhilsChan commented 2 years ago

Peer Review from Group 7

Members: Macy Chan @MacyChan, Philson Chan @PhilsChan, Mukund Iyer @miyer26, Pavel Levchenko @plevchen

Persona

Provide 3 pieces of constructive feedback that could help the app to be better suited to the scenario it was designed for Overall, the app is comprehensively designed for its usage. Here are some minor suggestions:

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

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

Are the plots selected appropiated 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

Luming-ubc commented 2 years ago

Hi @PhilsChan ,

Thanks for your constructive feedback. Just wanted to provide some of our responses to your feedback.

  1. As suggested, we have moved the demonstration section in the Readme file to the top to better serve its purpose.
  2. We have also include an Olympic logo as our Favicon. Hopefully, the user could get a better idea about our application in the Olympic games context.

I believe Cuthbert @cuthchow will address those feedback for the dashboard/application aspects in our next deployed version in Milestone 4. We look forward to it!