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

Fix CI failure due to incompatible qiskit packages #5661

Merged
merged 1 commit into from
Jul 1, 2022

Conversation

pavoljuhas
Copy link
Collaborator

@pavoljuhas pavoljuhas commented Jul 1, 2022

Both qiskit-aer and its dependency qiskit-terra attempt to write
qiskit/providers/__init__.py. If the version from qiskit-aer 0.9.1
wins it causes broken import of qiskit.providers.aer.
Here we update qiskit-aer to its latest version 0.10.4.

This fixes failing CI checks for Python 3.8 such as
https://github.com/quantumlib/Cirq/runs/7139878042.

Both qiskit-aer and its dependency qiskit-terra attempt to write
qiskit/providers/__init__.py.  If the version from qiskit-aer 0.9.1
wins it causes broken import of qiskit.providers.aer.

This fixes failing CI checks for Python 3.8 such as
https://github.com/quantumlib/Cirq/runs/7139878042.
@pavoljuhas pavoljuhas requested review from a team, vtomole and cduck as code owners July 1, 2022 03:03
@CirqBot CirqBot added the Size: XS <10 lines changed label Jul 1, 2022
@dstrain115 dstrain115 merged commit 0628174 into quantumlib:master Jul 1, 2022
@pavoljuhas pavoljuhas deleted the fix-qiskit-deps branch July 1, 2022 17:37
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
Both qiskit-aer and its dependency qiskit-terra attempt to write
qiskit/providers/__init__.py.  If the version from qiskit-aer 0.9.1
wins it causes broken import of qiskit.providers.aer.

This fixes failing CI checks for Python 3.8 such as
https://github.com/quantumlib/Cirq/runs/7139878042.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Size: XS <10 lines changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants