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

NBAgg figure management issues #4940

Closed
3 tasks
jenshnielsen opened this issue Aug 17, 2015 · 3 comments
Closed
3 tasks

NBAgg figure management issues #4940

jenshnielsen opened this issue Aug 17, 2015 · 3 comments
Milestone

Comments

@jenshnielsen
Copy link
Member

Remaining known issues after merging #4456

  • make sure figures get closed on kernel restart (also does not happen in 1.4.3)
  • still get a 'no such comm' error from closing figure programtically from python side
  • closing an animated figure triggers 'no such comm' error
@jenshnielsen jenshnielsen added this to the proposed next point release milestone Aug 17, 2015
@jenshnielsen
Copy link
Member Author

I tagged this as proposed next point release since:

I don't think these issues are critical and they are as far as I know also present in 1.4.3 but if possible they would be nice to fix in 1.5.0

@tritemio
Copy link

For the record issue #4457 may also be related (@jenshnielsen probably it is your second point).

@jenshnielsen
Copy link
Member Author

Have you checked with the master branch after the merge of #4456? I cannot reproduce any of those 2 issues anymore.

The last change that went into #4456 after your initial report makes sure that the figure is closed when removed from the DOM (I.e. not visible in the notebook) That takes care of issue one because there is no longer a hidden figure to plot into.

I tried opening several figures and closing them again and I see no sign of a memory leak. As expected the memory consumption goes up with more figures. It may not be immediately released when the figures are closed however explicitly doing a garbage collect seems to drop the memory consumption to the same level:

import gc 
gc.collect()

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

No branches or pull requests

3 participants