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

Yoshi/fix 170 #171

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Yoshi/fix 170 #171

wants to merge 13 commits into from

Conversation

yoshi-qc
Copy link
Contributor

@yoshi-qc yoshi-qc commented Feb 28, 2025

Description

Rigetti QPU Aspen-M-3 in Braket is deprecated.
For the new Rigetti QPU Ankka-3, "rigetti_provider_proverties_v2" is used, not v1. I edited braket.py for Ankka-3.

IonQ Harmony in Braket is also deprecated.
I edited the pytest file backend_test.py for IonQ Aria-2, Forte-1 and Rigetti Ankaa-3.

Related issues

#170

Checklist

  • I have performed a self-review of my code.
  • I have commented hard-to-understand parts of my code.
  • I have made corresponding changes to the public API documentation.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have updated the changelog with any user-facing changes.

Edited scripts for the schema "RIGETTI_SCHEM" in the function "_get_backend_info".
Deprecated IonQ "Harmony" and Rigetti "Aspen-M-3" in Braket.
Change them to IonQ "Aria-2", "Forte-1" and Rigetti "Ankaa-3".
@yoshi-qc yoshi-qc requested a review from cqc-alec as a code owner February 28, 2025 07:30
@cqc-alec
Copy link
Collaborator

One of the test failures is a timeout but the other looks like it's something broken. Are you looking into this?

yoshi-qc added 3 commits March 1, 2025 00:47
Rigetti QPU Ankaa-3 lacks the nodes 42 and 48. For the Ankaa-e, the option "disable_qubit_rewiring" sets to True, that is, no qubit rewiring is allowed.
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.

2 participants