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

feat: ankaa2 pulse #615

Merged
merged 23 commits into from
Aug 26, 2024
Merged

feat: ankaa2 pulse #615

merged 23 commits into from
Aug 26, 2024

Conversation

peterkomar-aws
Copy link
Contributor

Issue #, if available:

Description of changes:

Example notebooks about pulse control and verbatim compilation now use Ankaa-2 instead of Aspen-M-3.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@peterkomar-aws peterkomar-aws requested a review from a team as a code owner August 21, 2024 15:22
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@yitchen-tim
Copy link
Contributor

yitchen-tim commented Aug 21, 2024

@peterkomar-aws I left comment in the ReviewNB. The comments does not appear in the PR. Let me know if you cannot see them. The notebook all look good! I propose to run the bring-up notebook with a different qubit. Also, there are two other notebooks that are affect by the pulse bug, so it would be good to rerun them after the bug fix.

Copy link
Contributor

@yitchen-tim yitchen-tim left a comment

Choose a reason for hiding this comment

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

Pulse notebook 3,4 are affected by a bug. Would be good to run again after fix

" \"aspen\": {\n",
" \"device_arn\": Devices.Rigetti.AspenM3,\n",
" \"ankaa\": {\n",
" \"device_arn\": Devices.Rigetti.Ankaa2,\n",
" \"qubit\": 4,\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

qubit 4 happens to have bad T2 time. Could you use another qubit (eg qubit 3) with higher T2 to make the Ramsey experiment look better?

@yitchen-tim yitchen-tim reopened this Aug 26, 2024
@speller26 speller26 merged commit dbf828c into main Aug 26, 2024
8 checks passed
@speller26 speller26 deleted the feature/ankaa2-pulse branch August 26, 2024 16:59
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.

4 participants