ChillLive / ChillLiveWallpaperProject

6 stars 0 forks source link

Battery consumption #4

Open tjsuominen opened 1 year ago

tjsuominen commented 1 year ago

To me, seems the app is using pretty much battery (my phone OP Nord, Android 12).

Over 15% and tops the battery consumption list on most times...

Anything could be done (code-wise) to optimize that?

Screenshot_2023-06-13-08-12-02-24_0ba066473b79d6e213a1f6f52505e2ee

ChillLive commented 1 year ago

Hey!

Oh my god, this is shocking!

Do you have an always on display or something where the wallpaper is visible at all time? I don't understand how it's possible otherwise e. The app is programmed to kill all process when not showing to the user.

I will double check some of the parameters, but this is not normal.

Is it from the 0.49?

On my side, I have 1% after 8h, which is something but acceptable for me.

2023年6月17日(土) 3:56 Timo Suominen @.***>:

To me, seems the app is using pretty much battery (my phone OP Nord, Android 12).

Over 15% and tops the battery consumption list on most times...

Anything could be done (code-wise) to optimize that?

[image: Screenshot_2023-06-13-08-12-02-24_0ba066473b79d6e213a1f6f52505e2ee] https://user-images.githubusercontent.com/7660277/246505638-1fe65b67-547b-4089-a164-1ec3a457a344.jpg

— Reply to this email directly, view it on GitHub https://github.com/ChillLive/ChillLiveWallpaperProject/issues/4, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7ORBW2RX7RW7D4JIC2YMSDXLST6ZANCNFSM6AAAAAAZJTYW5A . You are receiving this because you are subscribed to this thread.Message ID: @.***>

tjsuominen commented 1 year ago

I have Always on display (on my OnePlus Nord), but that only shows notification (icon) from selected apps, not the wallpaper or anything. So that makes this weird... The only thing that I can imagine is about that for some reason, when a notification arrives the Chill wallpaper app also becomes "active" (without being visible) or some bug...?

Please also see attached activity details if that gives any further info to you.

Hope this gets fixed as this is a pretty major issue for me...

Screenshot_2023-06-17-09-45-42-15_0ba066473b79d6e213a1f6f52505e2ee

tjsuominen commented 1 year ago

The battery consumption goes 'wild' sometimes hogging a massive ~20% of battery power. Don't know how much that's related to my phone unlocking habits, but I could follow that too.

Have you been able to isolate this issue yet?

ChillLive commented 1 year ago

Hello !

I checked and I don't know why you are experiencing this...

It seems that you didn't have this problem before.

What I can propose you is to make a custom SDK with some newer features deactivated and send it by email for you to try.

What do you think? Let me know

2023年6月21日(水) 1:53 Timo Suominen @.***>:

The battery consumption goes 'wild' sometimes hogging a massive ~20% of battery power. Don't know how much that's related to my phone unlocking habits, but I could follow that too.

Have you been able to isolate this issue yet?

— Reply to this email directly, view it on GitHub https://github.com/ChillLive/ChillLiveWallpaperProject/issues/4#issuecomment-1599162811, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7ORBW6MQAUXKNVZSRLXKXDXMHIRPANCNFSM6AAAAAAZJTYW5A . You are receiving this because you commented.Message ID: @.***>

ChillLive commented 1 year ago

Hello Timo,

Good news, I think I found the problem. I had a loop that was never turned off even when the phone was locked. Hopefully it will solve your battery issue. Stay tuned!

On Wed, Jun 21, 2023 at 8:41 AM Chill @.***> wrote:

Hello !

I checked and I don't know why you are experiencing this...

It seems that you didn't have this problem before.

What I can propose you is to make a custom SDK with some newer features deactivated and send it by email for you to try.

What do you think? Let me know

2023年6月21日(水) 1:53 Timo Suominen @.***>:

The battery consumption goes 'wild' sometimes hogging a massive ~20% of battery power. Don't know how much that's related to my phone unlocking habits, but I could follow that too.

Have you been able to isolate this issue yet?

— Reply to this email directly, view it on GitHub https://github.com/ChillLive/ChillLiveWallpaperProject/issues/4#issuecomment-1599162811, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7ORBW6MQAUXKNVZSRLXKXDXMHIRPANCNFSM6AAAAAAZJTYW5A . You are receiving this because you commented.Message ID: @.***>

tjsuominen commented 1 year ago

That's brilliant news! Is that included in the 0.51 (21) release that just landed?

Will report soonest after tested in expected release for 1-2 days.

ChillLive commented 1 year ago

It should be solved with the 0.50, yes. 0.51 is solving another minor issue.

Let me know if your problem persist or not

2023年6月22日(木) 3:28 Timo Suominen @.***>:

That's brilliant news! Is that included in the 0.51 (21) release that just landed?

Will report soonest after tested in expected release for 1-2 days.

— Reply to this email directly, view it on GitHub https://github.com/ChillLive/ChillLiveWallpaperProject/issues/4#issuecomment-1601371649, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7ORBW65HQZBLSI76FXYV5LXMM4NZANCNFSM6AAAAAAZJTYW5A . You are receiving this because you commented.Message ID: @.***>

tjsuominen commented 1 year ago

It's got better (= less battery hog) with the 0.51 release, still at ~6% but definitely better. Seems there is still some optimization to be done (at least with OP phones).

Any idea implementing an optional debug log or something to collect data to identify possibly issues?

ChillLive commented 1 year ago

Hello Timo,

Thanks for the feedback.

Good news it went from 20 to 6%. It means that it identified one if the cause. I think I forgot another loop that was written the same way as the one I fixed last time. I will check this and get back to you.

If the problem isn't solved with the 0.52, then we'll see how to make a debug version.

Appart from that, I think that the app is stable enough to go V1 soon (after 0.52). I need to find places to advertise it to bring more users.

I'm working on the weather system. I'm sure you'll like it (check the screenshots)! But it's very early version.

2023年6月26日(月) 4:51 Timo Suominen @.***>:

It's got better (= less battery hog) with the 0.51 release, still at ~6% but definitely better. Seems there is still some optimization to be done (at least with OP phones).

Any idea implementing an optional debug log or something to collect data to identify possibly issues?

— Reply to this email directly, view it on GitHub https://github.com/ChillLive/ChillLiveWallpaperProject/issues/4#issuecomment-1606240302, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7ORBW7KRVKWIYR54ZLIGO3XNCJFVANCNFSM6AAAAAAZJTYW5A . You are receiving this because you commented.Message ID: @.***>

ChillLive commented 1 year ago

Hello Timo,

As I expected, I found another loop that wasn't shut down properly when locking the phone.

I fixed it and released the version 0.52. I think this was the last broken loop. It started from the release that brought the new animations (contrails/shooting stars), so v0.49 I think.

Let me know how it goes with this new version.

Thanks for your support!

2023年6月26日(月) 8:47 Chill @.***>:

Hello Timo,

Thanks for the feedback.

Good news it went from 20 to 6%. It means that it identified one if the cause. I think I forgot another loop that was written the same way as the one I fixed last time. I will check this and get back to you.

If the problem isn't solved with the 0.52, then we'll see how to make a debug version.

Appart from that, I think that the app is stable enough to go V1 soon (after 0.52). I need to find places to advertise it to bring more users.

I'm working on the weather system. I'm sure you'll like it (check the screenshots)! But it's very early version.

2023年6月26日(月) 4:51 Timo Suominen @.***>:

It's got better (= less battery hog) with the 0.51 release, still at ~6% but definitely better. Seems there is still some optimization to be done (at least with OP phones).

Any idea implementing an optional debug log or something to collect data to identify possibly issues?

— Reply to this email directly, view it on GitHub https://github.com/ChillLive/ChillLiveWallpaperProject/issues/4#issuecomment-1606240302, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7ORBW7KRVKWIYR54ZLIGO3XNCJFVANCNFSM6AAAAAAZJTYW5A . You are receiving this because you commented.Message ID: @.***>

tjsuominen commented 1 year ago

Installed 0.52 today, will report back tomorrow. What screenshots, with weather? Where? The ones on github are the old ones?

tjsuominen commented 1 year ago

Mailed you details yesterday. Time for debugging?

ChillLive commented 1 year ago

Hello!

Thanks for your quick feedback. Recently I'm super busy with work and have very little time to develop the app, unfortunately... Slow progress overall, but still working on it when I find time.

Based on your email yesterday, I just put log in every single methods to track which one could be still running in background. So far everything is quiet when the phone is locked.

I need to check more. Also, I know that behavior can be different depending on the device. Maybe your device is not calling some of the methods. I know it happens sometimes.

I can make a debug menu and send you an experimental version if you want, but first I need to find some time to create it

2023年7月5日(水) 6:56 Timo Suominen @.***>:

Mailed you details yesterday. Time for debugging?

— Reply to this email directly, view it on GitHub https://github.com/ChillLive/ChillLiveWallpaperProject/issues/4#issuecomment-1620782928, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7ORBW6XDXYDT26LLFGVV7TXOSGRBANCNFSM6AAAAAAZJTYW5A . You are receiving this because you commented.Message ID: @.***>

tjsuominen commented 1 year ago

Rgr that, thx for explaining. Keep up the great work to finalize your work as first release. It's ready when it's ready :)

tjsuominen commented 1 year ago

Any news on progress? :)