deprecation: IonQ Harmony #619
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
| @@ -133,11 +133,6 @@ | |||
| ] | |||
There was a problem hiding this comment.
| @@ -133,11 +133,6 @@ | |||
| ] | |||
There was a problem hiding this comment.
| @@ -104,10 +104,10 @@ | |||
| "outputs": [], | |||
There was a problem hiding this comment.
Here
{'arn:aws:braket:us-west-1::device/qpu/rigetti/Aspen-M-3': {'shots': 141, 'tasks': {'CREATED': 1}}, 'arn:aws:braket:us-east-1::device/qpu/ionq/Harmony': {'shots': 141, 'tasks': {'CREATED': 1}}}
Reply via ReviewNB
| @@ -96,17 +96,17 @@ | |||
| "print('\\n') \n", | |||
There was a problem hiding this comment.
Should be Ankaa-2 and Aria-1?.
Quantum Task Summary
{'arn:aws:braket:us-west-1::device/qpu/rigetti/Aspen-M-3': {'shots': 2000, 'tasks': {'COMPLETED': 2}}, 'arn:aws:braket:us-east-1::device/qpu/ionq/Harmony': {'shots': 1000, 'tasks': {'COMPLETED': 1}}}
Reply via ReviewNB
| @@ -262,7 +262,9 @@ | |||
| "outputs": [ | |||
There was a problem hiding this comment.
Line #5. # device_arn = Devices.Rigetti.AspenM3
Should we be removing this as well?
Reply via ReviewNB
| @@ -230,9 +230,9 @@ | |||
| "outputs": [ | |||
There was a problem hiding this comment.
|
Are we also planning to replace Aspen-M3 with Ankaa-2 everywhere? |
That'll be another PR, as it's more involved; this one only makes changes in examples where Harmony appears. |
Sounds good, just wanted to make sure that we don't miss out changes to any of our integ_tests as part of the arn changes for AspenM3 in this PR. |
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.