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

Pyqtgraph automatically store ref to the last 100 plots (configurable) #662

Merged
merged 4 commits into from
Jul 3, 2017

Conversation

jenshnielsen
Copy link
Collaborator

@jenshnielsen jenshnielsen commented Jun 30, 2017

Fixes #647

Currently pyqt graph plots may close spontaneously if you don't keep a reference to the plot around. This creates a deque as a Class parameter in QtPlot that automatically keeps the last n (default 100) plots in reference. The number of plots to keep can be configured. In my tests 100 plots of a (100,100) heatmap is roughly 1 GB of memory so this should be safe enough

Currently on top of #661 so that should be merged first
@QCoDeS/core

Copy link
Contributor

@WilliamHPNielsen WilliamHPNielsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks swell.

@jenshnielsen jenshnielsen merged commit ab60e0f into microsoft:master Jul 3, 2017
giulioungaretti pushed a commit that referenced this pull request Jul 3, 2017
Author: Jens Hedegaard Nielsen <[email protected]>

    Pyqtgraph automatically store ref to the last 100 plots (configurable) (#662)
Dominik-Vogel pushed a commit to Dominik-Vogel/Qcodes that referenced this pull request Aug 9, 2017
microsoft#662)

* Protect against crashed qt remote processes

* WIP trying to store refs to plots

* Store last 100 qt plots in a deque

* make max num plots a configurable
peendebak pushed a commit to VandersypenQutech/Qcodes that referenced this pull request Aug 11, 2017
microsoft#662)

* Protect against crashed qt remote processes

* WIP trying to store refs to plots

* Store last 100 qt plots in a deque

* make max num plots a configurable
@jenshnielsen jenshnielsen deleted the pyqt_store_refs branch October 25, 2024 11:56
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

Successfully merging this pull request may close these issues.

PyQtgraph windows are automatically closed if no reference is kept to the plot
2 participants