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.

Sign up for GitHub

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

Jump to bottom

qiskit.test is going to be removed in qiskit 1.0 #145

Closed
1ucian0 opened this issue Oct 11, 2023 · 3 comments · Fixed by #507
Closed

qiskit.test is going to be removed in qiskit 1.0 #145

1ucian0 opened this issue Oct 11, 2023 · 3 comments · Fixed by #507
Assignees
@frankharkins
Labels
content 📄
Milestone
24-02-15 Qiskit 1.0

Comments

@1ucian0
Copy link
Member

1ucian0 commented Oct 11, 2023

Qiskit exposes test utilities as part of its deliverable package. This is not consider good practice and its removal is a long overdue issue. Qiskit teams's intention is to clean that up for qiskit 1.0 (planned to be released on Jan 31st, 2024). The module will be available in the 0.* versions (EoL, mid 2024) and raise a deprecation warning.

The documentation uses qiskit.test in, at least, the following file:
https://github.com/Qiskit/documentation/blob/main/docs/api/qiskit-ibm-runtime/runtime_service.md?plain=1#L33

Consider stop importing qiskit.test (the IBM specific FakeProviders are moving to IBM packages).
@javabster javabster added this to the 24-02-13 Qiskit 1.0 milestone Dec 4, 2023
@frankharkins
Copy link
Member

@1ucian0 is this true for qiskit.test.reference_circuits? If so, do you know where that's moving?

@1ucian0
Copy link
Member Author

1ucian0 commented Dec 11, 2023

yes, they are getting moved to test/utils/reference_circuits.py, which is not part of the package. Do you need them for something?

@frankharkins
Copy link
Member

We're using it but we can easily switch it out, thanks!

@frankharkins frankharkins mentioned this issue Dec 11, 2023
Merged
github-merge-queue bot pushed a commit that referenced this issue Dec 11, 2023
@frankharkins
Remove use of qiskit.test (#507)
206ff0e 
Closes #145. That issue also refers to
`docs/api/qiskit-ibm-runtime/runtime_service.md` but we can't fix that
as it's generated from the runtime repo (should be covered by
Qiskit/qiskit-ibm-runtime#1244).
frankharkins added a commit to frankharkins/documentation that referenced this issue Jul 22, 2024
@frankharkins
Remove use of qiskit.test (Qiskit#507)
240e2e7 
Closes Qiskit#145. That issue also refers to
`docs/api/qiskit-ibm-runtime/runtime_service.md` but we can't fix that
as it's generated from the runtime repo (should be covered by
Qiskit/qiskit-ibm-runtime#1244).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@frankharkins frankharkins

Labels
content 📄
Projects
Docs Planning
Archived in project
Milestone
24-02-15 Qiskit 1.0
Development

Successfully merging a pull request may close this issue.

Remove use of qiskit.test Qiskit/documentation
4 participants
@1ucian0 @javabster @abbycross @frankharkins