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

[Bug] Memory leak in WidgetState #3062

Open
Antti-Palola opened this issue Apr 22, 2024 · 0 comments
Open

[Bug] Memory leak in WidgetState #3062

Antti-Palola opened this issue Apr 22, 2024 · 0 comments
Labels
type: bug 🐞 Errors in functionality

Comments

@Antti-Palola
Copy link
Contributor

Antti-Palola commented Apr 22, 2024

Bug description

Instantiating for example a plane widget and properly removing the widget and its manager with delete leaks memory.

Steps to reproduce

  • Instantiate a widget
  • Delete widget
  • Widget state leaks memory

Detailed Behavior

Loaded widgets three times, notice the groups of 3 distinct size, which are all different different states when loading the same view three times.

Screenshot 2024-04-22 at 09 54 02

Cause would seem to be

subscriptions.push(model.nestedStates[i].onModified(publicAPI.modified));
that pushes a callback in unbindState

Expected Behavior

No memory leaks

Environment

  • vtk.js version: main of 22.4.2024 (21534d1)
  • Browsers: Chrome
  • OS: Mac OS 14.3.
@Antti-Palola Antti-Palola added the type: bug 🐞 Errors in functionality label Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug 🐞 Errors in functionality
Projects
None yet
Development

No branches or pull requests

1 participant