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

Release0.13.3 #2052

Merged
merged 9 commits into from
Feb 6, 2024
Merged

Release0.13.3 #2052

merged 9 commits into from
Feb 6, 2024

Conversation

doichanj and others added 9 commits February 6, 2024 11:25
* fix test_aer_statevector

* fix daily test

* remove test case test.terra.states.test_aer_statevector.TestAerStatevector.test_number_to_latex_terms because of deprecation

* add release note and remove temporal on push
For the previous few releases the GPU wheel builds have failed to
upload. The root cause of this issue was that an earlier CI stage was
deleting on the locally cached docker images to free up space for
installing CUDA and all the GPU build requirements in the test VM.
Removing these docker images prevented the upstream action we run to
publish the wheels from working as it relied on having a docker image
available that couldn't be fetched from the default image repository.
This commit fixes this issue for future releases by ensuring we don't
delete the local docker cache in the GPU publish jobs. For the 0.13.2
releases this was manually applied and run to publish the wheels and
this PR is applying the fix for future releases.
* fix test cases for Qiskit 0.45.2

* format

* revert  testtools.TestCase to unittest.TestCase
* Remove remaining diagonal

* remove unused import transpile, replace isometry, uc to Isometry, UCGate

* format
* Add private primitive attributes removed by qiskit

* wrong url

* call private submit method

* Make submission compatible with qiskit <=0.45

* Skip Operator test is qiskit >=1.0

* grr lint
* Resolve CI dependencies in a single step and add qiskit==1.0.0rc1 job

This modifies the dependency-resolution steps to do their job in a
single step; Aer should have no dependencies pinning Qiskit, so we can
use the preferable form more eagerly.

This also adds a test run to the PR CI for Qiskit 1.0.0rc1 (or rc2, etc)
that always runs, so Aer doesn't regress.

Co-authored-by: Luciano Bello <bel@zurich.ibm.com>

* Fix test suite to work with Qiskit<1.0 and >=1.0

This commit *does not* touch the primitives, which appear to be broken
at a relatively basic level with Qiskit 1.0 - hopefully the primitives
team can fix that separately.

This modifies the test suite to work with both Qiskit 0.46 and 1.0.0rc1
simultaneously.  Unfortunately, some of the new-style fake backends
aren't in 0.46, so I just put in some import switches for my ease here.
A few tests, particularly in the noise model sections, were asserting
specific things about the properties/calibrations of Terra's fake
backends, which was quite fragile already, so in those particular cases
I replaced them with a manually constructed `Target` with properties
entirely under Aer's control.

* Run black

---------

Co-authored-by: Luciano Bello <bel@zurich.ibm.com>
Copy link
Member

@jakelishman jakelishman left a comment

Choose a reason for hiding this comment

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

This looks good to me, thanks! It'll be good to get Aer 0.13.3 out today.

@jakelishman jakelishman added the automerge This PR will automatically merge once its CI has passed label Feb 6, 2024
@mtreinish mtreinish merged commit e3f68bd into Qiskit:stable/0.13 Feb 6, 2024
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge This PR will automatically merge once its CI has passed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants