-
-
Notifications
You must be signed in to change notification settings - Fork 310
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
Samsung warns AndrOBD wakes often, and wants me to deep sleep it. #258
Comments
Thanks foryour detailled report. Did you ever try to do that test without any plugin activated? Thx |
Sounds like the plugin architecture needs to change. That the plugins should wait for a "start" message from the main app, and sleep otherwise. Though I've never tried to design one myself, and don't know about Android app limitations (I'm sure there are many). I just disabled the 2 plugins from the plugin manager, and will wait to see if I get any other messages. If it's the plugin(s) causing the issue, then shouldn't the message be about the plugin app instead? To change their behavior, versus the main app's. I haven't seen another message today, so I almost wonder if I hadn't closed the app out correctly before. And the issue is more about "app that shouldn't be running or doing anything was". That it's not quite sleeping as it could/should. Or maybe it won't warn me again since it already did, and this isn't a 'new' problem to the phone. I don't know much about that battery optimization feature, beyond being told to disable it for the particular app I'd first saw this for (to let it run). I'll look into it more later. |
Noticed the same behavior. In Settings->Development Options->Logging Level has to be set to minimum level of As far as I can tell, logging is turned on here at line 571 but never turned off when app is closed. Current log file at Sample of repeated lines after app is closed but NOT forced stopped: |
The event you see here is the event of screen refresh timer. However, just to make you aware: |
- timer.schedule on app.resume rather than app.start - timer.cancel on app.pause rather than app.close
Test version (V2.6.10) with corresponding fixes is available here: https://t.me/AndrOBD_dev/257 |
Describe the bug
Without actively using the application (haven't ran it for days) I'm seeing warnings about excessive "wakes" from the built in Battery features of my Samsung A71 phone (SM-A716V). I did try to use your app for the first time a few days ago, and I tried my best to close out of it fully at the end.
I'm sure I could work around the issue, or use the suggested setting (which would prevent the app from running unless I view it), but I thought I'd pass along the message in case it's something you can fix.
My understanding is the phone can't go into deep sleep (saving lots of battery when idle) if ANY application wakes it. Sure, if I was actively using something in AndrOBD I'd want it to wake, but I haven't run it for days now and never did see data come out of my adapter with it (Torque Lite worked though).
To Reproduce
"Extend your battery life
AndrOBD is waking up frequently and using too much battery. Put it in deep sleep to extend your battery life."
Expected behavior
No warnings, even when using default "Optimized" mode for the app (versus needing me to manually pick either "Unrestricted" or "Restricted" to prevent these today, and maybe lose battery life or app functionality)
Important details on your environment:
The text was updated successfully, but these errors were encountered: