Skip to content

Update AQT Backend#1

Merged
jbrixon merged 14 commits intomainfrom
update-aqt
Feb 5, 2024
Merged

Update AQT Backend#1
jbrixon merged 14 commits intomainfrom
update-aqt

Conversation

@jbrixon
Copy link
Copy Markdown
Collaborator

@jbrixon jbrixon commented Jan 24, 2024

No description provided.

@jbrixon jbrixon added the bug Something isn't working label Jan 24, 2024
@jbrixon jbrixon requested a review from fg-aqt January 24, 2024 09:34
@jbrixon jbrixon self-assigned this Jan 24, 2024
@jbrixon
Copy link
Copy Markdown
Collaborator Author

jbrixon commented Jan 24, 2024

@fg-aqt can you check the docs please? As discussed there are a few mentions of signing up via the Gateway Portal which are no longer relevant. Cheers!

@jbrixon jbrixon requested a review from airwoodix January 24, 2024 11:59
wilfried-huss
wilfried-huss reviewed Jan 24, 2024
View reviewed changes
Comment thread cirq-aqt/cirq_aqt/aqt_sampler.py Outdated
@fg-aqt
Copy link
Copy Markdown

fg-aqt commented Jan 25, 2024 •
edited
Loading

@fg-aqt can you check the docs please? As discussed there are a few mentions of signing up via the Gateway Portal which are no longer relevant. Cheers!

I updated the access.md and pushed the changes. I noticed that the functionality to retrieve the available workspaces with the respective resources is missing. Can you please add this?

I also did some preliminary updates of the getting_started instructions, we need to work some more on these.

airwoodix
airwoodix approved these changes Jan 26, 2024
View reviewed changes
Comment thread cirq-aqt/cirq_aqt/aqt_sampler.py Outdated
Comment thread cirq-aqt/cirq_aqt/aqt_sampler.py
Comment thread cirq-aqt/cirq_aqt/aqt_sampler_test.py Outdated
@jbrixon
Copy link
Copy Markdown
Collaborator Author

jbrixon commented Jan 29, 2024

I updated the access.md and pushed the changes. I noticed that the functionality to retrieve the available workspaces with the respective resources is missing. Can you please add this?

Is in with latest commit

@jbrixon jbrixon marked this pull request as ready for review January 29, 2024 11:34
fg-aqt
fg-aqt approved these changes Jan 29, 2024
View reviewed changes
Comment thread cirq-aqt/cirq_aqt/aqt_sampler.py
fg-aqt
fg-aqt requested changes Jan 30, 2024
View reviewed changes
Comment thread cirq-aqt/cirq_aqt/aqt_device.py Outdated
@jbrixon jbrixon merged commit 86f5eab into main Feb 5, 2024
@jbrixon jbrixon deleted the update-aqt branch February 5, 2024 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wilfried-huss wilfried-huss wilfried-huss left review comments

@fg-aqt fg-aqt fg-aqt approved these changes

+1 more reviewer

@airwoodix airwoodix airwoodix approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@jbrixon jbrixon

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jbrixon @fg-aqt @airwoodix @wilfried-huss