Skip to content

Commit ad29fa8

Browse files
authored
Fix Rigetti check for Aspen family device kind (#6734)
* Sync with new API for checking device family in qcs-sdk-python, Ref: rigetti/qcs-sdk-rust#463 in isa.pyi * Require qcs-sdk-python-0.20.1 which introduced the new family API Fixes #6732
1 parent d74e0dc commit ad29fa8

File tree

3 files changed

+7
-4
lines changed

3 files changed

+7
-4
lines changed

cirq-rigetti/cirq_rigetti/aspen_device.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ def __init__(self, isa: Union[InstructionSetArchitecture, Dict[str, Any]]) -> No
6666
else:
6767
self.isa = InstructionSetArchitecture.from_raw(json.dumps(isa))
6868

69-
if self.isa.architecture.family != Family.Aspen:
69+
if not Family.is_aspen(self.isa.architecture.family):
7070
raise UnsupportedRigettiQCSQuantumProcessor(
7171
'this integration currently only supports Aspen devices, '
7272
f'but client provided a {self.isa.architecture.family} device'

cirq-rigetti/cirq_rigetti/aspen_device_test.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -253,10 +253,10 @@ def test_rigetti_qcs_aspen_device_repr(qcs_aspen8_isa: InstructionSetArchitectur
253253
def test_rigetti_qcs_aspen_device_family_validation(qcs_aspen8_isa: InstructionSetArchitecture):
254254
"""test RigettiQCSAspenDevice validates architecture family on initialization"""
255255
non_aspen_isa = InstructionSetArchitecture.from_raw(qcs_aspen8_isa.json())
256-
non_aspen_isa.architecture.family = Family.NONE
256+
non_aspen_isa.architecture.family = Family.new_none()
257257

258-
assert (
259-
non_aspen_isa.architecture.family == Family.Aspen
258+
assert Family.is_aspen(
259+
non_aspen_isa.architecture.family
260260
), 'ISA family is read-only and should still be Aspen'
261261

262262

cirq-rigetti/requirements.txt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,4 @@
11
pyquil>=4.11.0,<5.0.0
2+
3+
# TODO - remove once pyquil requires qcs-sdk-python >= 0.20.1
4+
qcs-sdk-python>=0.20.1

0 commit comments

Comments
 (0)