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

Change alertMissedReading stepValue from 5 to 1 #353

Open
bjornoleh opened this issue Jan 18, 2025 · 0 comments
Open

Change alertMissedReading stepValue from 5 to 1 #353

bjornoleh opened this issue Jan 18, 2025 · 0 comments

Comments

@bjornoleh
Copy link
Contributor

I would suggest changing the alertMissedReading stepValue from 5 to 1.

With 5 minute increments, the alert can be triggered at the same moment a new reading appears. It would be better to allow triggering at times such as 6, 11, 16, 21 minutes instead of 5, 10, 15 or 20 minutes. This will reduce false alerts, and give the user four minutes until next reading is inbound to figure out if it is relevant to consider moving the receiving phone closer to the person who is wearing the sensor (typically a kid who left the phone in another room).

<<< StepperRow("alertMissedReading") { row in
        row.title = "Time"
        row.cell.stepper.stepValue = 5

Possibly other stepValue = 5 could be changed similarly. Either by setting it to 1, or forcing n+1 as the criterion.

Image

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

1 participant