Skip to content

Promote curve analysis deprecations from pending to current#1529

Merged
nkanazawa1989 merged 1 commit intoqiskit-community:mainfrom
wshanks:promote-curve-pending-deprecations
Mar 7, 2025
Merged

Promote curve analysis deprecations from pending to current#1529
nkanazawa1989 merged 1 commit intoqiskit-community:mainfrom
wshanks:promote-curve-pending-deprecations

Conversation

@wshanks
Copy link
Copy Markdown
Collaborator

@wshanks wshanks commented Mar 7, 2025

The functions in qiskit_experiments.curve_analysis.utils and methods
of ScatterTable that had previously been marked as pending deprecation
have been promoted to full deprecation status.

Additionally, some helper methods in
qiskit_experiments.curve_analysis.utils that had help replace the
deprecated functions have been documented as public.

Also, some methods on QiskitExperimentsTestCase that had been marked
as pending deprecation have been removed. QiskitExperimentsTestCase is
not part of the public API so these methods could be removed without a
deprecation cycle.

@wshanks
Copy link
Copy Markdown
Collaborator Author

wshanks commented Mar 7, 2025

This PR depends on #1526.

The functions in `qiskit_experiments.curve_analysis.utils` and methods
of `ScatterTable` that had previously been marked as pending deprecation
have been promoted to full deprecation status.

Additionally, some helper methods in
`qiskit_experiments.curve_analysis.utils` that had help replace the
deprecated functions have been documented as public.

Also, some methods on `QiskitExperimentsTestCase` that had been marked
as pending deprecation have been removed. `QiskitExperimentsTestCase` is
not part of the public API so these methods could be removed without a
deprecation cycle.
@wshanks wshanks force-pushed the promote-curve-pending-deprecations branch from 63a72e6 to 8327f46 Compare March 7, 2025 06:16
Copy link
Copy Markdown
Collaborator

@nkanazawa1989 nkanazawa1989 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nkanazawa1989 nkanazawa1989 added this pull request to the merge queue Mar 7, 2025
Merged via the queue into qiskit-community:main with commit 84611eb Mar 7, 2025
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.

2 participants