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

Tests and docstrings for autocalibration #319

Merged
merged 12 commits into from
May 2, 2023
Merged

Conversation

andrea-pasquale
Copy link
Contributor

This PR implements tests and docstrings for all the calibration routines used for autocalibration.
Tests are currently performed with the dummy platform.
Tests are passing locally but they will fail here because we should use qiboteam/qibolab#341

Checklist:

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

@stavros11
Copy link
Member

I did not review thoroughly, just wanted to say good job for adding the test and nice to see that it's so compact! 👏

def test_data_acquisition(action):
"""Test data acquisition for all routines using dummy"""
task = Task(action)
print(task)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is probably for debugging, just remember to remove.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh yes thank you.
Actually I wanted to check because even with the dummy platform some routines (especially allXY) may take a very long time due to qiboteam/qibolab#323.

@andrea-pasquale
Copy link
Contributor Author

I did not review thoroughly, just wanted to say good job for adding the test and nice to see that it's so compact! clap

Maybe there is a better way to do it. For the time being I'm just putting everything in a runcard :)

Copy link
Contributor

@Edoardo-Pedicillo Edoardo-Pedicillo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank @andrea-pasquale, LGTM, only one small comment

src/qibocal/protocols/characterization/ramsey.py Outdated Show resolved Hide resolved
@andrea-pasquale
Copy link
Contributor Author

@stavros11 let me know if you want to have a look at tests directly in this PR.
Otherwise I can merge it to #279 and we can have a look there directly.

@andrea-pasquale andrea-pasquale merged commit 67e46f7 into split_data_fit May 2, 2023
@andrea-pasquale andrea-pasquale deleted the tests branch May 2, 2023 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants