Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Include heart rate (pulse) information in BP measurement view #40

Open
asorici opened this issue Sep 17, 2018 · 0 comments
Open

Include heart rate (pulse) information in BP measurement view #40

asorici opened this issue Sep 17, 2018 · 0 comments
Assignees

Comments

@asorici
Copy link
Collaborator

asorici commented Sep 17, 2018

Why

For a medical professional, the heart rate is very important TOGETHER with the systolic and diastolic values ... because of many things - including that a higher heart rate will (although not linearly) produce an overly high blood pressure. Thus, if the heart rate is higher than usual, the blood pressure might not be relevant to look at ... and thus, we need to see the sys, dia and pulse together

What

  • include another bar, along with systolic and diastolic ones for heart_rate (since values are in the same ballpark, the graph should normally not get skewed)
  • if the graph becomes too dense, display history for last 5 days, instead of 7
  • to identify pulse readings that come from a BP measurement, use the device and timestamp fields in the JSON to do the filtering (they must have the same value as for a BP measurement)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants