-
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
Deprecate visualization code #11636
Deprecate visualization code #11636
Conversation
One or more of the the following people are requested to review this:
|
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.
Thanks for looking at this, Hunter. Could you add the removal_timeline
argument I put in the review comment to all the decorators touched by this PR, and revert the removals for the 0.46 branch?
@deprecate_arg("reverse_bits", since="0.22.0", package_name="qiskit-terra") | ||
def get_bit_reg_index(circuit, bit, reverse_bits=None): | ||
def get_bit_reg_index(circuit, bit): |
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.
These deprecations can stay in place through 0.46 - we'll remove everything in one fell swoop in 1.0 instead, because we're trying to keep 0.46 as close to being "0.45 with more warnings" as possible.
Since we're looking at them, it'd be good to include a removal_timeline="in the Qiskit 1.0 release"
on all the deprecate_arg
/deprecate_func
calls in this PR.
Summary
Following #10989, but for 0.46.
Details and comments
Set
pending=False
fordata
argument forplot_histogram
incounts_visualization.py
.Remove code and deprecation for
reverse_bits
arguments forget_bit_reg_index
andget_condition_label_val
incircuit/_utils.py
.