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

introduce default_passmanager to replace generate_preset_pass_manager #12161

Closed
wants to merge 7 commits into from

Conversation

1ucian0
Copy link
Member

@1ucian0 1ucian0 commented Apr 9, 2024

Summary

This PR introduces default_passmanager which is a cleaner version of generate_preset_pass_manager, designated to replace transpile in 2.0

Notes:

@1ucian0 1ucian0 requested a review from a team as a code owner April 9, 2024 12:12
@qiskit-bot
Copy link
Collaborator

One or more of the the following people are requested to review this:

  • @Qiskit/terra-core

@1ucian0 1ucian0 added this to the 1.1.0 milestone Apr 10, 2024
@jakelishman jakelishman modified the milestones: 1.1.0, 1.2.0 Apr 18, 2024
@raynelfss raynelfss self-assigned this May 17, 2024
@1ucian0 1ucian0 modified the milestones: 1.2.0, 1.3.0 Jul 17, 2024
@raynelfss raynelfss removed their assignment Sep 18, 2024
@raynelfss raynelfss removed this from the 1.3.0 milestone Oct 29, 2024
@mtreinish
Copy link
Member

I'm going to close this PR as I don't think we're pursuing this direction anymore. We re-exported generate_preset_pass_manager from the root qiskit module and also set 2 as the default value for optimization level. So users can do:

from qiskit import generate_preset_pass_manager
generate_preset_pass_manager(backend)

or

from qiskit import generate_preset_pass_manager
generate_preset_pass_manager(backend.target)

which I think covers the use case this was trying to address which was to lower the barrier to constructing a preset pass manager using a good default.

@mtreinish mtreinish closed this Oct 30, 2024
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.

5 participants