Open aldojasb opened 2 years ago
Well done Group 29!! The app is simple, lightweight , functional and very easy to use. It is very intuitive on its own. We have few suggestions :
Persona
Provide 3 pieces of constructive feedback that could help the app to be better suited to the scenario it was designed for:
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
[x] Is the text (plot labels / titles / subtitles) easy to read?
Yes, the labels/title/subtitles were easy to read and understand
[x] Are the plots selected appropriated for the data? Can you think of other alternatives or improvements for the plots regarding interactivity/reactivity or plot selection?
Well, as per the porposal.md , there are lot other categories which are yet to be added, which in our view will make this app wholesome in helping new grads to decide whether or not the county is aptly suited to their needs or not. As of now, since weather and social category related plots have been included, in our view it might not give the whole picture about the place. In future, when Group 29 includes all the plots as per their mission it will give a clearer picture to the target audience.
Another minor suggestion would be to add tooltip on each line plot which gives us value about the county and temp details, interactivity can also be added to the plots based on each county. But these are just add-ons for more details and totally based on personal preferences.
Also the x-axis(month's label) instead of 1-12 , if Jan-Dec could be included. Not sure, if it will make the plots more crowded, but can be a good option to try out.
[x] Was the dashboard organized in a way that was easy to read (columns / rows) or you would suggest a change in this sense?
The bar plots in one row appears a bit squished, so may be these can be made horizontal.
[x] 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?
[x] Are the plots or other images described properly for the persona described?
Although the plots are very well organized, one change which the group can incorporate in future is to divide the all the plots which belong to the same category such as weather, social, economic into different tabs for better understanding and more interpretability.
Yes, the plots were described properly and were pretty self explanatory from the titles and axis labels
Backend
[x] 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
[x] Was the documentation easy to understand from the README.md how to use the app? How could the documentation be improved?
[x] 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
Yes, the documentation and steps in the README.md and instructions to use the app were very clear.
Yes, because of the simplicity of the app, it was very intuitive and one can easily use it without reading the documentation and just by looking at the gif.
We really loved how after adding the counties, the selected counties appeared beside the dropdown, which could be easily reset, this is something we have not added and would like to include in our app too so that the users can be aware of their selections.
Absolutely loved and really enjoyed reviewing your app since the target audience and persona matches a lot with our project. It also gave us more ideas about things that we can add to our own app. Once again Great work!!
DSCI532-peer-review
To complete by the group
proposal.md
file where it is the description of the persona and scenario: https://github.com/UBC-MDS/your_happy_place/blob/main/doc/proposal.mdTo complete by the group reviewing:
Persona
Frontend / layout
Backend
Documentation
README.md
how to use the app? How could the documentation be improved?Indicate something you would like to include in your own app after reviewing this one