Skip to content

Add MultiTemporaryAND operation as Operation class#9379

Draft
Qottmann wants to merge 3 commits intocprodfrom
multitemporaryand3
Draft

Add MultiTemporaryAND operation as Operation class#9379
Qottmann wants to merge 3 commits intocprodfrom
multitemporaryand3

Conversation

@Qottmann
Copy link
Copy Markdown
Contributor

Context:

This operation is ubiquitous in many subroutines, makes sense to not have to re-code it

In #9321 tried to make MultiTemporaryAND inherit from ControlledOp, similarly to how MultiCtrontrolledX is constructed. Trying to avoid the same design flaws and construct the operation here by inheriting from Operation (more similar to what TemporaryAND is doing).

Description of the Change:

Benefits:

Simplifying, unifying and improving a lot of subroutines in the code base

Possible Drawbacks:

Another op/subroutine to maintain 🚧

Related GitHub Issues:

@github-actions
Copy link
Copy Markdown
Contributor

Hello. You may have forgotten to update the changelog!
Please edit doc/releases/changelog-dev.md with:

  • A one-to-two sentence description of the change. You may include a small working example for new features.
  • A link back to this PR.
  • Your name (or GitHub username) in the contributors section.

@Qottmann Qottmann requested review from KetpuntoG and dwierichs April 30, 2026 14:18
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.

1 participant