Closed mattStorer closed 5 months ago
after additional communications, it sounds like we may be making more changes to this low BP logic to coincide more closely with high BP logic. more discussions needed to nail down the precise details, TBD.
Here is my understanding of how it should behave. We will need confirmation from @davedorr9.
First, check for 2 low BPs both within 14 days. If found:
If there are not 2 low BPs within 14 days, but the most recent BP is low (no matter how old):
As discussed on Thursday, this does not necessarily solve the problem for people who have low blood pressure all the time because their medication is working "too well". They could still get caught in a cycle of being told to contact their care team over and over again. To help limit this, there was a proposal to lower the diastolic threshold to 50 instead of 60.
AE logic was previously only creating a Hypotension event if the two low BPs were HOME BPs. In order for logic to be consistent throughout the app, all BPs will be included in the logic.
presently, hypotension adverse events aren't being filtered when going to the recommendation engine to ensure that they're current (within the last month). this needs to happen. filtration logic should be added into AdverseEventService.getAdverseEvents just before hae is converted into an AdverseEvent and added to the list, just before this line:
https://github.com/OHSUCMP/coach/blob/master/src/main/java/edu/ohsu/cmp/coach/service/AdverseEventService.java#L55