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 visualization code #11636

Merged
merged 6 commits into from
Feb 1, 2024
Merged

Conversation

hunterkemeny
Copy link
Contributor

Summary

Following #10989, but for 0.46.

Details and comments

Set pending=False for data argument for plot_histogram in counts_visualization.py.

Remove code and deprecation for reverse_bits arguments for get_bit_reg_index and get_condition_label_val in circuit/_utils.py.

@hunterkemeny hunterkemeny requested a review from Cryoris January 24, 2024 20:26
@hunterkemeny hunterkemeny requested review from nonhermitian and a team as code owners January 24, 2024 20:26
@qiskit-bot
Copy link
Collaborator

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

@jakelishman jakelishman added this to the 0.46.0 milestone Jan 30, 2024
Copy link
Member

@jakelishman jakelishman left a 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?

Comment on lines 200 to 199
@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):
Copy link
Member

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.

@jakelishman jakelishman added Changelog: Deprecation Include in "Deprecated" section of changelog mod: visualization qiskit.visualization labels Jan 31, 2024
qiskit/visualization/circuit/_utils.py Outdated Show resolved Hide resolved
qiskit/visualization/circuit/_utils.py Outdated Show resolved Hide resolved
qiskit/visualization/circuit/_utils.py Outdated Show resolved Hide resolved
qiskit/visualization/circuit/_utils.py Outdated Show resolved Hide resolved
qiskit/visualization/circuit/_utils.py Outdated Show resolved Hide resolved
qiskit/visualization/circuit/_utils.py Outdated Show resolved Hide resolved
qiskit/visualization/circuit/_utils.py Outdated Show resolved Hide resolved
jakelishman
jakelishman previously approved these changes Feb 1, 2024
@jakelishman jakelishman enabled auto-merge February 1, 2024 12:11
@jakelishman jakelishman added this pull request to the merge queue Feb 1, 2024
Merged via the queue into Qiskit:stable/0.46 with commit 3d3edd0 Feb 1, 2024
11 checks passed
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 mod: visualization qiskit.visualization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants