-
Notifications
You must be signed in to change notification settings - Fork 7
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
added web visualization of multiple data reports #156
added web visualization of multiple data reports #156
Conversation
for more information, see https://pre-commit.ci
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## alvaro/fast_calibrate_qubit_states_pr #156 +/- ##
=========================================================================
- Coverage 18.76% 18.47% -0.30%
=========================================================================
Files 13 13
Lines 943 958 +15
=========================================================================
Hits 177 177
- Misses 766 781 +15
Flags with carried forward coverage won't be shown. Click here to find out more.
|
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
src/qibocal/calibrations/characterization/resonator_spectroscopy.py
Outdated
Show resolved
Hide resolved
src/qibocal/calibrations/characterization/qubit_spectroscopy.py
Outdated
Show resolved
Hide resolved
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.
Thanks for implementing this @DavidSarlle.
You can find below a few comments.
A possible suggestion maybe not related to this PR. When the qq-compare
command is failing it would be cool to show the error on the same page instead of showing it in a new web page:
src/qibocal/calibrations/characterization/resonator_spectroscopy.py
Outdated
Show resolved
Hide resolved
Implemented |
…m/qibocal into david/report-concatenate
for more information, see https://pre-commit.ci
…m/qibocal into david/report-concatenate
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.
Thanks for the updates @DavidSarlle.
This PR is ready to merge in my opinion.
This branch implements the possibility to select multiple saved reports to visualize data in the same webpage using qq-live. Unlike qq-compare, this function will show multiple saved reports without checking if the selected reports are comparable.
Checklist: