Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Peer Feedback from rhj641, kwf929, vrp628 #5

Open
AlfredJarlner opened this issue Apr 21, 2024 · 0 comments
Open

Peer Feedback from rhj641, kwf929, vrp628 #5

AlfredJarlner opened this issue Apr 21, 2024 · 0 comments

Comments

@AlfredJarlner
Copy link

The most elegant solution:
Your heat maps are very informative and makes it easier to understand. We also liked your explanations of your data outcomes, which made it easy to understand for non opioid experts.

Hardest part to understand:
The only hard part to understand is your preliminary description and your map with pharmacy chains. Your code with the pharmacy chains, is a little hard to understand in the sense that you don't explain why you have separated them in two groups.

What could be better documented:
We don't have any critique on the documentation other that what commented on in the last comment.

Idea of improvement:
If you are not, directly using the preliminary data description for the reader, then an improvement will be to move it to a .py file instead because otherwise it just creates noise.

Idea for extension:
Maybe show data about the effects on the hospital system or homelessness in order to further highlight the problem

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant