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

high memory usage #128

Closed
arthurEspooner opened this issue Jan 12, 2023 · 7 comments
Closed

high memory usage #128

arthurEspooner opened this issue Jan 12, 2023 · 7 comments

Comments

@arthurEspooner
Copy link

while doing testing for #126 i realized that this plugin was using a lot of memory after a few hours. i couldn't figure out if a specific button was doing it, or the fact i had 32 buttons at one time on an XL using this plugin. these are various sensors and media controls. memory usage will climb over the course of a few hours and i've seen it as high as 40(!) GB.

i've had this plugin uninstalled for 2 days now and this is not happening. i'm going to do some more testing to see if it was a specific entity/etc causing it.

@arthurEspooner
Copy link
Author

arthurEspooner commented Jan 15, 2023

i reinstalled the HA plugin and readded about a 1/3 of the overall buttons i was using with it. i made it as basic as possible, no custom calls or even icons. it's been solid for 2 days now, no disconnects, instant response, and also no #126 or #71 anymore.

@arthurEspooner
Copy link
Author

spoke too soon, it's back now with same behavior. commands are delayed, resource usage is high, then eventually it stops responding while resource usage keeps climbing until you close steam deck.

shame, guess i need to restart the stream deck app hourly via a script or something. seems like multiple people are having these problems spread across a few different issues.

@cgiesche
Copy link
Owner

Hmpf. I will add 20 fast updating sensors and see what happens...

@cgiesche
Copy link
Owner

cgiesche commented Jan 25, 2023

F***
image
few seconds later
image
few seconds later
image

Well. I have no idea, how JavaScript memory management works. I am doomed....

cgiesche added a commit that referenced this issue Jan 27, 2023
@cgiesche
Copy link
Owner

I found the memory leak and fixed it.

@cgiesche
Copy link
Owner

@arthurEspooner A new release (https://github.com/cgiesche/streamdeck-homeassistant/releases/tag/2.2.0) is on its way to the elgato store. Maybe you want to install the version directly to check if some of your issues are fixed.

(I noticed that memory usage is still high(er) than normal if you have connected debugger or chrome dev tools but this seems to be ok, as memory usage goes back to normal as soon as dev tools are closed.)

@arthurEspooner
Copy link
Author

yep, it's fixed, thank you! i wasn't following this and since plugins auto-update i didn't realize this got deployed. i haven't seen this for a few weeks though and i was checking consistently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants