Skip to content
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

Deprecation of MCMT in favor of MCMTGate #13584

Open
wants to merge 3 commits into
base: stable/1.4
Choose a base branch
from

Conversation

1ucian0
Copy link
Member

@1ucian0 1ucian0 commented Dec 19, 2024

The Multiple-Control-Multiple-Target in qiskit.circuit.library.generalized_gates.MCMT was pending for deprecation in #13150 . This PR moves it to deprecated.

@1ucian0 1ucian0 requested a review from a team as a code owner December 19, 2024 17:38
@qiskit-bot
Copy link
Collaborator

One or more of the following people are relevant to this code:

  • @Cryoris
  • @Qiskit/terra-core
  • @ajavadia

@1ucian0 1ucian0 added this to the 1.4.0 milestone Dec 19, 2024
@1ucian0 1ucian0 added the Changelog: Deprecation Include in "Deprecated" section of changelog label Dec 19, 2024
@Cryoris Cryoris added on hold Can not fix yet and removed on hold Can not fix yet labels Dec 20, 2024
@Cryoris
Copy link
Contributor

Cryoris commented Dec 20, 2024

I think we want to deprecate the circuit library objects for 1.4 and 2.0, but might not want to remove them for 2.0. Even though this PR only changes from PendingDeprecationWarning to DeprecationWarning, we probably should discuss what the exact plan is 🙂

@1ucian0 1ucian0 removed the on hold Can not fix yet label Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: Deprecation Include in "Deprecated" section of changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants