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

Animations consume MUCH CPU #16

Open
miccl opened this issue Dec 30, 2018 · 5 comments
Open

Animations consume MUCH CPU #16

miccl opened this issue Dec 30, 2018 · 5 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@miccl
Copy link

miccl commented Dec 30, 2018

Hi,
while using your useful module, I noticed that MM uses a lot of CPU. After some digging, I found out, that the cool animated icons of your module are the reason. If the animations (currentIcon and weatherForecastIcon) are activated, the CPU usage ascends by > 50%).
Do you have any idea why this error occurs?
I deactivated the animations for now, but would love to use them again.
Thanks in advance

@nkl-kst
Copy link
Owner

nkl-kst commented Jan 2, 2019

Hi Michael,

thanks for opening this issue. I experienced a similiar problem when using the animated icons. In my case, the whole MagicMirror app crashes after some hours (approximately 24 +/- x hours). I'm pretty sure it's the Electron browser who crashes, because the "backend" is still available (e.g. the MMM-Remote-Control plugin). In the pm2 logs I can see an error message like "out of memory [...]" without any specific stacktrace or anything else.

I tried to debug the problem with Chrome DevTools, investigating heap dumps and memory timelines, but without any success. I ended up using static icons too, but this is unsatisfying 😔

If you have any further informations, please let me know. I really want to solve this problem, because I like the animated icons on the mirror. But this will probably take some time to debug it again and I can't predict any result ...

@nkl-kst nkl-kst added bug Something isn't working help wanted Extra attention is needed labels Jan 2, 2019
@miccl
Copy link
Author

miccl commented Jan 2, 2019

Hi Nikolas,

yep. You describe exactly what I experienced as well. Sadly I only saw the same bland error message.
But if I have a free minute and find something, I will let you know.

Greets from a fellow metropolian 😄

@AleksanderG
Copy link

Have you tried to use a new Electron version with the Chromium updates?

@nkl-kst
Copy link
Owner

nkl-kst commented Oct 31, 2022

Hi Aleksander, I update my MM regularly but didn't try the animated icons for a long time. Even static icons aren't displaye currently so I think the whole icons are broken anyway. Maybe I should give it a try ...

@AleksanderG
Copy link

The static icons work fine for me with 2.21.0.

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

No branches or pull requests

3 participants