search

OpenSeizureDetector / Android_Pebble_SD

The main OpenSeizureDetector Android App, that is published on the Android Play Store.
http://openseizuredetector.org.uk
GNU General Public License v3.0
9 stars 11 forks source link

v4.2.x Enabling Puls Alarm using PineTime causes Alarm to trigger all the time #150

Closed seaside1 closed 3 months ago

seaside1 commented 3 months ago

Steps to reproduce:

1) Launch app with a pine time 2) Under Settings-> Seizure Detector under section "Simple Heart Rate Alarm Setting" enable Puls Alarm (Heart rate measurement frozen warning not enabled, Alarm vi zeropuls not enabled) (Using default values of min puls: 40 max puls 150) 3) Now going back to the app or retarting it by "Force stop" the alarm will trigger no matter what thresholds are set under puls min / max

Another question, Why are there two " Puls Alarms" under settings? One under Algorithm settings and another one under Simple Heart Rate Alarm Setting? Maybe the one under algorithm should be removed?

jones139 commented 3 months ago

Thanks- I'll have a look this evening- I thought it was working OK!

jones139 commented 3 months ago

Well, I might have found the issue, and it is in the PineTime firmware - although we switch on the heart rate monitor when the pinetime boots, the indicated heart rate was zero. If I go into the heart rate app (swipe up from main screen and press the heart icon), and switch the heart rate monitor off and then on again, it starts registering the correct heart rate. You can see the heart rate history in V4.2.x by swiping the graph at the bottom of the main screen to the right. I'll try re-booting the pinetime a few times and see if I can find out why the heart rate value does not start automatically.

For your other question, the reason the 'Enable Simple Heart Rate alarm' checkbox appears twice is to make it easier to just scan down and enable and disable algorithms, as they are all at the top - the detail for each algorithm is further down and we put the 'Enable' checkbox with the detailed settings too.

jones139 commented 3 months ago

As this is really a bug on the Pinetime Firmware, I have created a new issue (https://github.com/OpenSeizureDetector/PineTimeSD/issues/13) and will close this one.

jones139 commented 3 months ago

For some reason github is showing this as completed trying to close it again with a different closure tag - please refer to OpenSeizureDetector/PineTimeSD#13