Skip to content

Conversation

@tanujkhattar
Copy link
Collaborator

Fixes #5302

@tanujkhattar tanujkhattar requested review from a team, cduck and vtomole as code owners May 21, 2022 00:04
@CirqBot CirqBot added the size: S 10< lines changed <50 label May 21, 2022
Copy link
Collaborator

@95-martin-orion 95-martin-orion left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for this fix!

@tanujkhattar tanujkhattar added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label May 23, 2022
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label May 23, 2022
@tanujkhattar tanujkhattar removed automerge Tells CirqBot to sync and merge this PR. (If it's running.) front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. labels May 23, 2022
@tanujkhattar tanujkhattar merged commit 3e3d6f7 into master May 23, 2022
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
…s transformer (quantumlib#5386)

* Ignore operations on more than 10 qubits in drop_negligible_operations transformer

* Assert same circuits

Co-authored-by: Cirq Bot <craiggidney+github+cirqbot@google.com>
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
…s transformer (quantumlib#5386)

* Ignore operations on more than 10 qubits in drop_negligible_operations transformer

* Assert same circuits

Co-authored-by: Cirq Bot <craiggidney+github+cirqbot@google.com>
@pavoljuhas pavoljuhas deleted the fix_drop_negligible branch January 22, 2025 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: S 10< lines changed <50

Projects

None yet

Development

Successfully merging this pull request may close these issues.

optimize_for_target_gateset OOMs on large CircuitOps

3 participants