-
Notifications
You must be signed in to change notification settings - Fork 315
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
Feature/qt plot #290
Feature/qt plot #290
Conversation
Current coverage is 88.00% (diff: 100%)
|
I forgot to mention that this plot is now a QtWidget, which allows to integrate it in other qt guis easily... |
Just tested the PR and it works fine on my system. I took the liberty of adding part of my own additions to |
Great idea, what system are you using? |
Some minor issues:
|
Good points! As for QtPy, I never tried it, but it might be a good idea. I remember though having problems with PySide, and PyQtgraph wont work with PyQt5 I think. But yes, that shouldnt stop us from doing it right. |
System is ubuntu 16.04, but our measurement system is windows, so we'll need that one as well (I can test next week). I agree using qcodes style is the right thing to do. |
And thanks for testing this! |
@MerlinSmiles The dockarea The commit for pyqtgraph is here: |
Also To solve the issues one can install a more recent version of pyqtgraph:
|
Deze regel: https://github.com/qdev-dk/Qcodes/pull/290/files#diff-4e71a3e969fcdacead1806f2d8a6afd7R140 |
@eendebakpt I can confirm. This solved it for me in python 3.4.5 |
@MerlinSmiles 🍻 +review ? |
@giulioungaretti @MerlinSmiles Any news on this PR? |
''' | ||
Get the default title for a data_array, which is used as the subplot | ||
title. It is just the DataSet location and the array_id. | ||
A custom title can be passed using **kwargs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MerlinSmiles kwargs? Am I missing something?
@MerlinSmiles which version of py qt graph are you using ? the last release is 0.9.10 |
actually, let's all wait until pyqtgraph/pyqtgraph#377 is closed and v 0.10.0 is released ! |
I am closing this PR and deleting the branch after testimonies from Giulio and Merlin that this is nukeable. |
Fixes the Qt part of #266
Fixes #259 I think
Fixes the Qt part of #217
Fixes #34 since
setGeometry
is now directly available and_repr_png_
existsFixes #289 I implemented it here too...
Changes proposed in this pull request:
QtPlot.add()
QtPlot.add()
(flip x, y axes, and transpose z)name
and array_id, in the case where name exists. however, location is nice to later relate screenshots to the right dataset.update
toupdate_data
inbase.py
so it wont interfere withupdate()
from theQwidget
I'm leaving for a holiday now, but it would be great if someone could check this over, and give some input
This PR is mostly meant to fix some of the urgent issues I came across, there is still a lot to do but for that we need to refactor the whole plotting system, which seems somewhat complicated right now. Make it more modular and allow for easy additions...
@giulioungaretti @alexcjohnson @AdriaanRol @peendebak @nataliejpg