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

Fix Classification Parameters closes #305 #308

Merged
merged 3 commits into from
May 3, 2023

Conversation

aorgazf
Copy link
Member

@aorgazf aorgazf commented Apr 22, 2023

The PR fixes the bug introduced in the calculation of the rotation angle and threshold of the calibrate_qubit_states_fit function.
With this fix, AllXYs work again.

Checklist:

  • Reviewers confirm new code works as expected.
  • Tests are passing.
  • Coverage does not decrease.
  • Documentation is updated.

@aorgazf aorgazf added the bug Something isn't working label Apr 22, 2023
@aorgazf aorgazf self-assigned this Apr 22, 2023
@codecov
Copy link

codecov bot commented Apr 22, 2023

Codecov Report

Merging #308 (81cf97f) into main (598e862) will increase coverage by 0.04%.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #308      +/-   ##
==========================================
+ Coverage   37.85%   37.89%   +0.04%     
==========================================
  Files          42       42              
  Lines        2800     2797       -3     
==========================================
  Hits         1060     1060              
+ Misses       1740     1737       -3     
Flag Coverage Δ
unittests 37.89% <0.00%> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/qibocal/fitting/methods.py 72.69% <0.00%> (+0.76%) ⬆️

@Edoardo-Pedicillo
Copy link
Contributor

At this point I think you can drop this two lines

iq_state0_translated = iq_state0 - origin
iq_state1_translated = iq_state1 - origin

@andrea-pasquale
Copy link
Contributor

At this point I think you can drop this two lines

iq_state0_translated = iq_state0 - origin
iq_state1_translated = iq_state1 - origin

+1

@andrea-pasquale
Copy link
Contributor

I'll go ahead and merge this PR.

@andrea-pasquale andrea-pasquale merged commit 40d4db4 into main May 3, 2023
@andrea-pasquale andrea-pasquale deleted the alvaro/fix_classification_parameters branch May 3, 2023 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants