-
Notifications
You must be signed in to change notification settings - Fork 14
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
Refactor IcarusQ drivers #661
Merged
Merged
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
a1cd446
icarusq instrument base
sorewachigauyo ee23a2d
driver update for icarusq alpha 21
sorewachigauyo 987784d
IcarusQ alpha21.1 support
sorewachigauyo 5108d97
Merge branch 'main' of https://github.com/qiboteam/qibolab into icaru…
sorewachigauyo e54c675
single sweeper implementation
sorewachigauyo 86d2537
Merge branch 'main' of https://github.com/qiboteam/qibolab into icaru…
sorewachigauyo 6726066
Update icarusq drivers
sorewachigauyo 90ac98a
icarusq driver pylint
sorewachigauyo f1f390a
icarusq driver pylint fix
sorewachigauyo ee8d4a7
Merge branch 'main' of https://github.com/qiboteam/qibolab into icaru…
sorewachigauyo c366a82
IcarusQ sweeper implementation and import cleanup
sorewachigauyo 1b1dcbb
Merge branch 'main' of https://github.com/qiboteam/qibolab into icaru…
sorewachigauyo ed58ea6
Use qibolab pulse envelopes and add deps
sorewachigauyo 4f2e6b6
IcarusQ update for Qibocal and sweepers
sorewachigauyo c08af5d
Update icarusq instrument methods
sorewachigauyo b87ea2c
Update icarusq rfsoc readout and sweeper
sorewachigauyo 2581f9b
Merge branch 'main' of https://github.com/qiboteam/qibolab into icaru…
sorewachigauyo bb0f7a1
Add sampling rate for IcarusQ
sorewachigauyo a449052
Add sampling rate property to IcarusQ RFSoC
sorewachigauyo 497e5a7
Rework IcarusQ connect to be in line with other instruments
sorewachigauyo 4a0be6f
Fix change for pylint
sorewachigauyo 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
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.
This is directly uploading the attenuation value to the instrument and similarly the getter grabs it for the instrument. This is opposite approach to what is used in the other instruments where we keep a cache of parameters even before connecting. Not a big issue as this instrument is quite simple, but it could cause complication if you want to set (eg. load from runcard) values before connecting.