pilot51 / voicenotify

Android app that speaks notifications
https://voicenotify.app
Apache License 2.0
150 stars 54 forks source link

Heavy Sensor Usage #30

Closed edwardlthompson closed 10 years ago

edwardlthompson commented 10 years ago

52m and 42s of sensor usage. What sensor is being used and why?

pilot51 commented 10 years ago

Only the accelerometer for shake-to-silence, but it is designed (perhaps poorly) to only be active while a notification is being spoken. For the next release, I plan to fix some reported issues with shake-to-silence, which would hopefully resolve any excess sensor usage.

edwardlthompson commented 10 years ago

I may be mistaken. When I checked earlier in the day it was the highest user of the sensors. You may be able to ignore this. I turned off the shake to silence feature. If i see it happen again I will be sure to grab a screenshot for you.

2014-04-17 04 03 10

pilot51 commented 10 years ago

I'll leave this open for now, as a reminder if nothing else, until I fix up the shake-to-silence code. Of course, please comment if you notice significant sensor usage again.

pilot51 commented 10 years ago

Issue #24 is fixed and turned out to be caused by acceleration values being kept and used between notifications, not because the sensor was active between them. I'll close this anyway, but feel free to reopen if you see this issue again.