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

Remove qiskit-terra and qiskit-ibm-provider when testing against qiskit main #1372

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

wshanks
Copy link
Collaborator

@wshanks wshanks commented Jan 24, 2024

Other packages that depend on qiskit<1.0 will pull in qiskit and qiskit-terra together and qiskit-terra will conflict with qiskit main, so we remove it before installing qiskit main.

qiskit-ibm-provider 0.8.0 (current stable) tries to import qiskit.extensions for a basic import qiskit_ibm_provider which does not exist in qiskit main any more. qiskit-ibm-provider registers a qiskit plugin which causes it to get automatically imported under some conditions, triggering a warning in jupyter execute blocks which then cause a strict sphinx build to fail.

…it main

Other packages that depend on qiskit<1.0 will pull in qiskit and
qiskit-terra together and qiskit-terra will conflict with qiskit main,
so we remove it before installing qiskit main.

qiskit-ibm-provider 0.8.0 (current stable) tries to import
`qiskit.extensions` for a basic `import qiskit_ibm_provider` which does
not exist in qiskit main any more. qiskit-ibm-provider registers a
qiskit plugin which causes it to get automatically imported under some
conditions, triggering a warning in jupyter execute blocks which then
cause a strict sphinx build to fail.
Copy link
Collaborator

@coruscating coruscating left a comment

Choose a reason for hiding this comment

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

Works for me locally, thanks. I do get

ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
qiskit-dynamics 0.4.4 requires qiskit<1.0, but you have qiskit 1.0.0 which is incompatible.

after installing qiskit, but I don't think there's an easy way to get around that right now and it doesn't fail the test.

@wshanks wshanks added this pull request to the merge queue Jan 25, 2024
@wshanks
Copy link
Collaborator Author

wshanks commented Jan 25, 2024

Right, we just have to live with that error from pip. It should warn about qiskit-ibm-experiment as well once a new release is tagged.

Merged via the queue into qiskit-community:main with commit e648bc5 Jan 25, 2024
11 checks passed
@wshanks wshanks deleted the qiskit-main-remove-terra branch January 25, 2024 16:31
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