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

Improved plots: temperatures, stage lines and annotations, large legends outside axes, regex sample selection #7

Merged
merged 8 commits into from
Jan 20, 2022

Conversation

cgevans
Copy link
Owner

@cgevans cgevans commented Jan 20, 2022

Checklist:

  • Documentation added or revised for changes.
  • Tests added for new features and fixes, or revised for changes. (Good enough...)
  • Real-system tests run manually, and pass (if applicable).

@cgevans cgevans changed the title Plot temperatures Improved plots: temperatures, stage lines and annotations, large legends outside axes, regex sample selection Jan 20, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jan 20, 2022

Codecov Report

Merging #7 (1fbf509) into main (8e3fcfa) will increase coverage by 0.09%.
The diff coverage is 65.34%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
+ Coverage   57.69%   57.79%   +0.09%     
==========================================
  Files          19       19              
  Lines        3725     3793      +68     
  Branches      754      773      +19     
==========================================
+ Hits         2149     2192      +43     
- Misses       1399     1412      +13     
- Partials      177      189      +12     
Impacted Files Coverage Δ
src/qslib/experiment.py 59.06% <63.04%> (+0.07%) ⬆️
src/qslib/normalization.py 48.42% <85.71%> (+2.96%) ⬆️
src/qslib/protocol.py 74.28% <100.00%> (+0.03%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8e3fcfa...1fbf509. Read the comment docs.

@cgevans cgevans linked an issue Jan 20, 2022 that may be closed by this pull request
6 tasks
@cgevans cgevans removed a link to an issue Jan 20, 2022
6 tasks
@cgevans cgevans self-assigned this Jan 20, 2022
@cgevans cgevans added the enhancement New feature or request label Jan 20, 2022
@cgevans cgevans closed this Jan 20, 2022
@cgevans cgevans reopened this Jan 20, 2022
@cgevans cgevans marked this pull request as ready for review January 20, 2022 14:41
@cgevans cgevans merged commit 51b1946 into main Jan 20, 2022
@cgevans cgevans deleted the plot_temperatures branch January 20, 2022 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants