-
Notifications
You must be signed in to change notification settings - Fork 25
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
[FR] Background Support #23
Comments
Might make instrumentation testing more difficult though. Gotta get those unit tests working. |
How this should work?
With this solution would be possible to get rid of WakeLock but trade off is that notification will be always visible until tracking is stopped. If this solution is OK then I can take this task |
Hmm... having an always visible notification is fine I think. Constantly show a heads-up like dashboard with the current battery % and maybe speed, if it isn't too notification heavy. Your method might be just fine, my thoughts when I saw this PR was wrapping the services in an actual Android Service (startService() etc) and passing the data via intents or http://square.github.io/otto/ Goal at the end of the day is to keep the app running, logging, etc - it's annoying that I'll be going, look down at the app, then will be disconnected. And would be great to see all the stats in the lockscreen notification, stretch goal :) |
I've been using the master build for a couple of days. It's looking good, a few hiccup that I'm trying to track down and replicate - mainly related to power management. If I turn off the screen, BLE disconnects, and also seems to disconnect even with screen on at certain times. Logs are showing the BLE stack sending the disconnect... Sounds like power management stuff (Doze etc), adb commands can test power management triggers etc as well like app standby (https://source.android.com/devices/tech/power/mgmt) Oh, the battery and speed gauge will freeze at times, stats etc still update. Likely due to the updates coming from different channels. I'm still tracking this one down. |
Ok. If possible can you fill separate issue tickets for 100% identified problems please? I will try to solve them one by one. |
A background service would be a nice feature to add. So you can pull up maps, texting, camera, or whatever and still be logging.
The text was updated successfully, but these errors were encountered: