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

feat: update to qiskit 1.3 #57

Merged
merged 3 commits into from
Dec 19, 2024
Merged

feat: update to qiskit 1.3 #57

merged 3 commits into from
Dec 19, 2024

Conversation

gschurck
Copy link
Contributor

@gschurck gschurck commented Dec 6, 2024

No description provided.

@gschurck gschurck self-assigned this Dec 6, 2024
@gschurck gschurck force-pushed the migrate-qiskit-1.3 branch 2 times, most recently from 44bd2ea to 2fbcea4 Compare December 10, 2024 15:38
@gschurck gschurck marked this pull request as ready for review December 11, 2024 09:14
@gschurck gschurck requested review from sbrieuc and alxthm December 11, 2024 09:14
sbrieuc
sbrieuc previously approved these changes Dec 17, 2024
@sbrieuc sbrieuc dismissed their stale review December 17, 2024 09:24

Seems one issue still persists

@gschurck gschurck force-pushed the migrate-qiskit-1.3 branch 3 times, most recently from c8cab9b to 85d83b4 Compare December 19, 2024 15:54
Some tests had to be updated for Qiskit 1.3.
This new Qiskit version changes the default optimization_level of
transpile() from 1 to 2. This change enables other
transpilation passes like VF2Layout and ApplyLayout passes that shuffle
the qbits randomly in the circuit. As we need a deterministic and
reproducible behavior for the tests, we fix the seed in transpile().
@gschurck gschurck merged commit 21a7ffc into main Dec 19, 2024
13 checks passed
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.

3 participants