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

Update flux script #981

Closed
wants to merge 40 commits into from
Closed

Update flux script #981

wants to merge 40 commits into from

Conversation

Edoardo-Pedicillo
Copy link
Contributor

Small refactoring of the flux script in #954. I have merged the main and #959 in script_coherence ( #954).

Checklist:

  • Reviewers confirm new code works as expected.
  • Tests are passing.
  • Coverage does not decrease.
  • Documentation is updated.
  • Compatibility with Qibo modules (Please edit this section if the current pull request is not compatible with the following branches).
    • Qibo: master
    • Qibolab: main
    • Qibolab_platforms_qrc: main

alecandido and others added 30 commits July 29, 2024 23:32
To be more explicit as an example and template
The drive frequency is now updated only when a detuning
is provided, which is one of the assumption of the fit.
When detuning is 0 we update T2 since we expect it to
be more reliable.
@Edoardo-Pedicillo Edoardo-Pedicillo marked this pull request as draft September 11, 2024 09:08
@Edoardo-Pedicillo Edoardo-Pedicillo changed the base branch from script_coherence to main September 11, 2024 09:12
@Jacfomg Jacfomg mentioned this pull request Sep 20, 2024
8 tasks
@andrea-pasquale
Copy link
Contributor

I'm closing this since all code for the qibocal paper is now available in https://github.com/qiboteam/qibocal-paper-data

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.

5 participants