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

Turn off accelerometer in night mode #2030

Open
1 task done
JustToby22 opened this issue Feb 27, 2024 · 3 comments
Open
1 task done

Turn off accelerometer in night mode #2030

JustToby22 opened this issue Feb 27, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@JustToby22
Copy link

Verification

  • I searched for similar issues (including closed issues) and found none was relevant.

Introduce the issue

In night mode, I realised that while the PineTime doesn't react to any accelerometer-based "display on" events, it does react to the "lower wrist" event.
I have to say, I haven't read the code, but it seems to me that the accelerometer is still active and reporting its state during night mode. Since many people, as I do, likely use Night mode to preserve power, it seems illogical to keep the accelerometer running while it's on.
One could argue that it would still be necessary for step counting, however. Personally, I'd take power savings over accurate step count while I'm trying to save power/go to sleep.

Preferred solution

Disable the accelerometer in night mode.

Version

1.14.0

@FintasticMan
Copy link
Member

FintasticMan commented Feb 27, 2024

I think that the accelerometer does indeed not turn off in night mode, but you wouldn't be able to tell by lower to sleep still working. This is because the accelerometer should only turn off when the screen is off.

Also, the accelerometer is able to track steps while in a low-power state, which is what we use when neither raise to wake nor shake to wake is enabled. This is the mode I think would be best to implement while sleep mode is on.

@FintasticMan FintasticMan added the bug Something isn't working label Feb 27, 2024
@L3br4nd
Copy link

L3br4nd commented Jun 19, 2024

What about people who want to use the watch for sleeptracking?
They would still need the accelerometer.
In that case there should be an option to keep it on.

@FintasticMan
Copy link
Member

The accelerometer stays awake if there is a device subscribed to the motion service. This means that if you're using an app on your phone to track sleep, it won't turn off. We'll make sure to not break that while fixing this bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants