-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Removed Deprecated VF2 Call Limit #11540
Removed Deprecated VF2 Call Limit #11540
Conversation
One or more of the the following people are requested to review this:
|
Pull Request Test Coverage Report for Build 7482563209
💛 - Coveralls |
releasenotes/notes/deprecation-passmanager-0.25-95eb9b45b517370a.yaml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks for doing this removal!
upgrade: | ||
- | | ||
Removed deprecated function :func:`~qiskit.transpiler.preset_passmanagers.common.get_vf2_call_limit` | ||
and its corresponding test. Instead, use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One more small point for release note style is that we ought to only mention user-facing changes, so in this case we can omit the bit about the tests being updated. Not a reason to hold up this PR, we can easily adjust this when we create and review the release notes PR during release 🙂.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good, I will keep this in mind the next time I write a release note, thanks! :-)
* deprecate 0.25 passmanager * revert * removed deprecated function get_vf2_call_limit * minor: lint/black * reno fix
Summary
Removed deprecated
get_vf2_call_limit
function and associated tests.Details and comments