Skip to content

Adding barriers between cycles#1199

Merged
vietphamngoc merged 7 commits into
mainfrom
qcvv-barriers
May 30, 2025
Merged

Adding barriers between cycles#1199
vietphamngoc merged 7 commits into
mainfrom
qcvv-barriers

Conversation

@vietphamngoc

@vietphamngoc vietphamngoc commented May 27, 2025

Copy link
Copy Markdown
Contributor

Adding barriers to prevent the compiler from reducing the circuits to Id.

@vtomole vtomole requested a review from dowusu-antwi May 28, 2025 15:34

@richrines1 richrines1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one request but otherwise lg!

Comment thread supermarq-benchmarks/supermarq/qcvv/base_experiment.py Outdated
@cdbf1

cdbf1 commented May 29, 2025

Copy link
Copy Markdown
Contributor

Thanks for this @vietphamngoc !

vietphamngoc and others added 2 commits May 29, 2025 18:47
Integrating Rich's suggestion

Co-authored-by: richrines1 <85512171+richrines1@users.noreply.github.com>

@richrines1 richrines1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lg % (optional) nits!

Comment thread supermarq-benchmarks/supermarq/qcvv/base_experiment_test.py Outdated
Comment thread supermarq-benchmarks/supermarq/qcvv/base_experiment_test.py Outdated
Comment thread supermarq-benchmarks/supermarq/qcvv/base_experiment.py Outdated
vietphamngoc and others added 2 commits May 30, 2025 15:36
Switching `TaggedOperation` with `with_tags`

Co-authored-by: richrines1 <85512171+richrines1@users.noreply.github.com>
Switching `TaggedOperation` with `with_tags`

Co-authored-by: richrines1 <85512171+richrines1@users.noreply.github.com>
@richrines1

Copy link
Copy Markdown
Contributor

@vietphamngoc is this ready to merge?

@vietphamngoc vietphamngoc merged commit 4b7f90c into main May 30, 2025
19 checks passed
@vietphamngoc vietphamngoc deleted the qcvv-barriers branch May 30, 2025 15:36
@vietphamngoc

Copy link
Copy Markdown
Contributor Author

Yup just did

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.

3 participants