-
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
RX90 calibration implementation #1044
Merged
Merged
Changes from all commits
Commits
Show all changes
81 commits
Select commit
Hold shift + click to select a range
14f80b4
first test for swittching from pi to pi-half
ElStabilini ec4b7ad
fixed pi half calibration sequence
ElStabilini 9f63c69
modified classes and methods for running rabi for RX90
ElStabilini 33db217
added option to save duration of RX90 pulse
ElStabilini 32cceda
started modifying rabi ef for using RX90
ElStabilini 7661361
modified pihalf_pulse to rx90 in parameters
ElStabilini d03f383
Update rabi.rst
ElStabilini d0c1228
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] e6a2395
set pi calibration as default and change rx90 dtype
ElStabilini 144f468
Saving modified documentation
ElStabilini c112748
fix ordering error
ElStabilini f083434
moved all pihalf_pulses in Rabi-Data class
ElStabilini f52765f
moved all pihalf_pulses in Rabi-Data class
ElStabilini efcb55a
fixed errors
ElStabilini 66f0b71
Update rabi.rst
ElStabilini 1f16f50
fixed amplitude_sequence and length_sequence inputs
ElStabilini f8e9021
Sync remote and local branches
ElStabilini a08cf3c
fixed acquisition error
ElStabilini 0f9ec82
fixed error
ElStabilini 6111a46
fix rabi length signal
ElStabilini 973b318
fix rabi length signal
ElStabilini 3164fd8
align True
ElStabilini 393d039
Update rabi.rst
ElStabilini 1742dc1
Add rabi_amplitude_rx90
ElStabilini 30dd98c
Update rabi.rst
ElStabilini f521abc
rename pi_half to rx90
ElStabilini bf70bf6
Sync local and remote branches
ElStabilini 349e312
force ef rabi protocol with pihalf
ElStabilini 07a62b8
Delete rabi_length_signal.yaml
ElStabilini 1873ccb
Update src/qibocal/protocols/rabi/utils.py
ElStabilini ef2e21a
Update src/qibocal/protocols/rabi/amplitude_frequency.py
ElStabilini 7deade0
Update src/qibocal/protocols/rabi/length_frequency.py
ElStabilini 222539d
Update src/qibocal/protocols/rabi/length_frequency.py
ElStabilini 62230c9
Update src/qibocal/protocols/rabi/length.py
ElStabilini 23ae111
Update src/qibocal/protocols/rabi/ef.py
ElStabilini 4dbcc34
Update src/qibocal/protocols/rabi/length.py
ElStabilini 3427756
Update update.py
ElStabilini 3c676ff
Update protocols.yml
ElStabilini df453df
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 92e0d71
Update pyproject.toml
ElStabilini 1f9d532
Update length_frequency.py
ElStabilini 31db8b7
Update ef.py
ElStabilini c579cbb
Update flipping.py
ElStabilini 64e5352
Update pyproject.toml
ElStabilini 3c4e0d9
Update poetry.lock
ElStabilini 04cf0f9
Update flipping.py
ElStabilini cf73084
modify amplitude_sequence and length_sequence for rx90, add flipping …
ElStabilini f6bfcaf
Update flipping.py
ElStabilini 454f5b5
Update flipping.py
ElStabilini 47a22b5
Update flipping.py
ElStabilini 96dd44e
Update flipping.py
ElStabilini 07a4bcd
Update flipping.py
ElStabilini e8cea9a
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 4052755
Update flipping.py
ElStabilini 12fedb5
fix FlippingResults and FlippingData class definition
ElStabilini b58e13a
fix globalbackends error: downgrade qibo
ElStabilini 32308e2
Update flipping.rst
ElStabilini 57904a1
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 3b6769d
fix error in _fit method
ElStabilini 29a1296
update documentation for flipping
ElStabilini 022240d
fix tests for rx90
ElStabilini 8b0995d
Update flipping.rst
ElStabilini 98bcf1d
modify plot function in utils to have the correct name of the pulse c…
ElStabilini 17ba606
'sync remote and local branch'
ElStabilini 7b907ae
improve code for printing pulse names on reports
ElStabilini 7660fde
fix compatibility with qibo v0.2.14
ElStabilini 2d8c8f7
fix conflict with pyproject.toml for branch 0.2
ElStabilini 1bac594
fix error in flipping sequence construction for rx90
ElStabilini af82fec
Merge branch '0.2' into pi_half
ElStabilini 228a02e
Update src/qibocal/protocols/flipping.py
ElStabilini 35c7178
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 8950008
Update src/qibocal/protocols/flipping.py
ElStabilini b90f887
fix test definition and improve code for flipping with rx90 gate
ElStabilini 23ee280
'sync remote and local fixes'
ElStabilini 01a4d1a
Update utils.py
ElStabilini c5bb28f
improve documentation for flipping and rabi protocols
ElStabilini f62bbc3
Update doc/source/protocols/rabi/rabi.rst
ElStabilini 832d0e9
create first RX90 of the sequence always using R.(theta=pi/2)
ElStabilini 05c2886
fix tabs
ElStabilini 478c864
sync remote and local update
ElStabilini 837cc08
Update src/qibocal/protocols/flipping.py
ElStabilini File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Instead of mentioning the experiment in the general section, and describe it only in the example, you may move this description up, and just cite here you're performing the pi/2 described above.
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.
Moreover, a description of the sequence (i.e. "same, but replace each pi with two pi/2") and a brief motivation may be appreciated. Then, you may cite this section in the flipping, instead of repeating it.
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.
Even a reference to the literature you consulted may be appreciated as well :)