-
Notifications
You must be signed in to change notification settings - Fork 301
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
Plotting / support complex numbers in plot_by_id #1592
Plotting / support complex numbers in plot_by_id #1592
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1592 +/- ##
=========================================
Coverage ? 72.22%
=========================================
Files ? 116
Lines ? 12459
Branches ? 0
=========================================
Hits ? 8999
Misses ? 3460
Partials ? 0 |
Codecov Report
@@ Coverage Diff @@
## master #1592 +/- ##
==========================================
+ Coverage 70.49% 70.55% +0.05%
==========================================
Files 123 123
Lines 14881 14921 +40
==========================================
+ Hits 10491 10528 +37
- Misses 4390 4393 +3 |
(and include a test that covers them both)
f561c22
to
cc70e67
Compare
@ThorvaldLarsen, you were one of the first people to request this feature. I'd be good to know if you think that this PR does the job, or whether we overlooked something. |
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.
Apart from minor comments, i think this is ready. Thanks for the example notebook!
@WilliamHPNielsen this looks great to me! Thanks for getting it done. |
So as to have one test per test case for the preparser
…n/Qcodes into plotting/complex_numbers
The notebook should use load_or_create_experiment and explicitly pass the experiment to the measurement |
... and pass Experiment object explicitly to Measurement object.
- improve description of plot_by_id behavior - get rid of divide-by-zero warning from the last plot - rerun notebook to ensure everything plots correctly
Done. plot_dataset() change I would defer to another PR. so that we change plot_by_id for plot_dataset in the whole qcodes at once. |
Support the plotting of complex numbers in
plot_by_id
. Still WIP.Changes proposed in this pull request:
Still pending:
plot_by_id
phase_and_mag
intomag_and_phase
@QCoDeS/core