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

Deprecate backend_props arg in generate_translation_pass_manager #13838

Closed
wants to merge 1 commit into from

Conversation

mtreinish
Copy link
Member

Summary

This commit deprecates the backend_props argument for the generate translation pass manager helper function. This argument was removed as part of cleaning up the transpiler interface in 2.0 to remove loose constraints and any tie to the BackendV1 models objects. However, the fact that this function was public was overlooked and a deprecation was not added ahead of time.

Details and comments

Fixes #13831

This commit deprecates the backend_props argument for the generate
translation pass manager helper function. This argument was removed as
part of cleaning up the transpiler interface in 2.0 to remove loose
constraints and any tie to the BackendV1 models objects. However, the
fact that this function was public was overlooked and a deprecation was
not added ahead of time.

Fixes Qiskit#13831
@mtreinish mtreinish added the Changelog: Deprecation Include in "Deprecated" section of changelog label Feb 13, 2025
@mtreinish mtreinish added this to the 1.4.0 milestone Feb 13, 2025
@mtreinish mtreinish requested a review from a team as a code owner February 13, 2025 16:49
@qiskit-bot
Copy link
Collaborator

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

  • @Qiskit/terra-core

@ElePT
Copy link
Contributor

ElePT commented Feb 13, 2025

I was going to extend a bit the message in #13719, the PR that originally deprecated the argument (the one that triggered #13831). Would you be ok with that? That would allow to have everything contained in a single reno.

@mtreinish mtreinish closed this Feb 13, 2025
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