Skip to content

cirq-aqt module not working anymore (due to breaking changes in the AQT API) #6379

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

Closed
fg-aqt opened this issue Dec 11, 2023 · 7 comments · Fixed by #6441
Closed

cirq-aqt module not working anymore (due to breaking changes in the AQT API) #6379

fg-aqt opened this issue Dec 11, 2023 · 7 comments · Fixed by #6441
Assignees
Labels
status/needs-agreed-design We want to do this, but it needs an agreed upon design before implementation

Comments

@fg-aqt
Copy link

fg-aqt commented Dec 11, 2023

The AQT service hosted at gateway-portal.aqt.eu has been deactivated and is replaced with a new solution by AQT. Usage of the current version of the cirq-aqt module is therefore no longer possible! Accordingly, the documentation at https://quantumai.google/cirq/hardware#aqt-hardware is outdated.

The new service offers improved access to AQT's quantum computers and can be used through the API described at https://arnica.aqt.eu/api/v1/docs.

The qiskit-aqt-provider may help as a reference for adapting cirq-aqt module to make use of the new API.

@dstrain115
Copy link
Collaborator

Adding AQT owners. Can you help maintain this integration please?

@tanujkhattar tanujkhattar added the triage/discuss Needs decision / discussion, bring these up during Cirq Cynque label Jan 3, 2024
@fg-aqt
Copy link
Author

fg-aqt commented Jan 8, 2024

Adding AQT owners. Can you help maintain this integration please?

@dstrain115: thanks for assigning @pschindler and @ma5x, but they are probably not the right persons to take care of this. We from AQT can help maintain this integration but it would be beneficial if the process is driven by the Cirq team. At the moment I am not able to tell when and who will be able to invest some effort into this, but we can assign the issue to me in the mean-time and preferably to someone suitable on your end.

@NoureldinYosri NoureldinYosri added status/needs-agreed-design We want to do this, but it needs an agreed upon design before implementation and removed triage/discuss Needs decision / discussion, bring these up during Cirq Cynque labels Jan 17, 2024
@dstrain115
Copy link
Collaborator

Hi @fg-aqt. Thanks for the update. We can definitely help with this integration, but we would strongly prefer that someone from the AQT drive the process. Fixing the integration after a breaking change in the AQT API definitely qualifies as maintaining the service. AQT folks are going to be more familiar with the changes that happen and will have more access to credentials to test the service. This is really something that has to be driven by the AQT team.

If there is anything we can do to make the code contribution process easier or any questions about . We also have a bi-weekly cirq cync that we would encourage someone to join that meeting.

@jbrixon
Copy link
Contributor

jbrixon commented Jan 24, 2024

Hi @dstrain115, sure. We're just finalising a fix internally and will submit a pull request shortly. You can happily assign this issue to me in the meantime.

Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 30 days

@fg-aqt
Copy link
Author

fg-aqt commented Feb 26, 2024

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 30 days

I don't have permissions to remove the label, can someone else please do it?

Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 30 days

dstrain115 pushed a commit that referenced this issue Apr 19, 2024
Adjusted the AQT backend so that it works with the new AQT Arnica API. We also updated the gateset to reflect what we support.
As we made an extension to the API, we were able to add a feature that lists the workspaces and resources that are available to a user. We also updated tests and documentation accordingly.

Fixes #6379
jselig-rigetti pushed a commit to jselig-rigetti/Cirq that referenced this issue May 28, 2024
Adjusted the AQT backend so that it works with the new AQT Arnica API. We also updated the gateset to reflect what we support.
As we made an extension to the API, we were able to add a feature that lists the workspaces and resources that are available to a user. We also updated tests and documentation accordingly.

Fixes quantumlib#6379
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/needs-agreed-design We want to do this, but it needs an agreed upon design before implementation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants