-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Possible memory leak in idle app #1401
Comments
Pretty sure this is the sinewave generator, which accidentally keeps references to unsubscribe functions. |
This is still an issue |
I left "My Items" running for ~30 minutes, and 805 kbytes of new memory was allocated (big blue line before is garbage collection): Here's the heap snapshot: |
Ran it again and saw ~350 kbytes of growth over ~30 minutes: @akhenry looks pretty minimal to me? |
@scottbell we have an automated test for this. Let's sync out of band |
Running it in dev mode will always leak some amount of memory. Both Webpack HMR and Chrome dev tools are known to leak memory. @johnhill2 Did you observe the memory leak locally or in a production environment? It's probably worth repeating this with a prod build (vanilla Open MCT is fine, doesn't need to be one of our deployments) so HMR isn't a factor. I bet those arrays are full of Webpack stuff. |
@akhenry production environment on hulk during the STM Devsims on root folder menu |
In production mode, memory was more flat: |
@scottbell within tolerances, yup. Let's close this for now. |
Steps to reproduce
Sawtooth shape is due to regular minor garbage collections. Additionally there is one full GC at the start triggered manually, and two full GCs (triggered twice manually for good measure) at the end. Despite the two full GCs heap allocation does not return to its original size.
The text was updated successfully, but these errors were encountered: