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

Add Python 3.9 support #1071

Merged
merged 7 commits into from
Dec 10, 2020
Merged

Add Python 3.9 support #1071

merged 7 commits into from
Dec 10, 2020

Conversation

mtreinish
Copy link
Member

@mtreinish mtreinish commented Dec 7, 2020

Summary

This commit adds official python 3.9 support to qiskit-aer. This
includes CI, release notes, and package metadata.

Details and comments

Fixes #1067

Upstream dependencies that need Python 3.9 support:

- [ ] osqp (sidestepped by skipping cvxpy tests for py39)

This commit adds official python 3.9 support to qiskit-aer. This
includes CI, release notes, and package metadata.

Fixes Qiskit#1067
@mtreinish
Copy link
Member Author

mtreinish commented Dec 7, 2020

It looks like python 3.9 builds work fine (and passes verify wheels), but there are some CI issues for the tests jobs around cvxpy and related on python 3.9 that are blocking those jobs.

mtreinish and others added 3 commits December 8, 2020 17:57
The assertNoLogs methods was built using a private class from python's
stdlib unittest library. This should never have been done as it's
explicitly marked as private. Accordingly in Python 3.9 this private
class has been removed and no longer exists. It turns out this method
was not used anywhere in all of qiskit (the same identical code was
removed from terra as part of Qiskit/qiskit#5189 in
Qiskit/qiskit@ed5155b).
Since the implementation is not sound and nothing uses it this commit
just removes the class and method.
@mtreinish
Copy link
Member Author

mtreinish commented Dec 9, 2020

One thing to note here is that since #1074 we're not trying to test the noise transformations on Python 3.9 anymore as we're no longer installing cvxpy outside of python 3.7 and 3.8. There were issues installing cvxpy because of scs with 3.9. I'm thinking we should document this in the release notes.

Copy link
Contributor

@vvilpas vvilpas left a comment

Choose a reason for hiding this comment

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

Looks good! Just a small typo.

@@ -140,6 +140,8 @@ stages:
python.version: '3.7'
Python38:
python.version: '3.8'
Python38:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Python38:
Python39:

Copy link
Member Author

Choose a reason for hiding this comment

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

I fixed this issue yesterday to unblock the azure runs: bb99ae1 I think you might have been looking at a stale diff.

Copy link
Contributor

Choose a reason for hiding this comment

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

Right! Sorry about that.

@@ -229,6 +231,8 @@ stages:
python.version: '3.7'
Python38:
python.version: '3.8'
Python38:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Python38:
Python39:

@vvilpas vvilpas merged commit 1dbb97e into Qiskit:master Dec 10, 2020
@mtreinish mtreinish deleted the add-3.9-support branch December 10, 2020 19:12
mtreinish added a commit to mtreinish/aqua that referenced this pull request Jan 14, 2021
Python 3.9.0 was released on 10-05-2020, this commits marks the support
of Python 3.9 in qiskit-ignis. It adds the supported python version in
the package metadata and updates the CI configuration to run test jobs
on Python 3.9. This follows on from Qiskit/qiskit#5189,
qiskit-community/qiskit-ignis#505, and Qiskit/qiskit-aer#1071
mtreinish added a commit to mtreinish/aqua that referenced this pull request Jan 14, 2021
Python 3.9.0 was released on 10-05-2020, this commits marks the support
of Python 3.9 in qiskit-aqua. It adds the supported python version in
the package metadata and updates the CI configuration to run test jobs
on Python 3.9. This follows on from Qiskit/qiskit#5189,
qiskit-community/qiskit-ignis#505, and Qiskit/qiskit-aer#1071
manoelmarques pushed a commit to qiskit-community/qiskit-aqua that referenced this pull request Jan 15, 2021
* Add support for Python 3.9

Python 3.9.0 was released on 10-05-2020, this commits marks the support
of Python 3.9 in qiskit-aqua. It adds the supported python version in
the package metadata and updates the CI configuration to run test jobs
on Python 3.9. This follows on from Qiskit/qiskit#5189,
qiskit-community/qiskit-ignis#505, and Qiskit/qiskit-aer#1071

* Add release note

* Remove 3.9 tutorial job

Jupyter on 3.9 doesn't have a good story yet, so just run tutorials on
3.6 and 3.8 for now.

* Update optional requirement system constraints

Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
@chriseclectic chriseclectic added the stable-backport-potential The issue or PR might be minimal and/or import enough to backport to stable label Jan 22, 2021
chriseclectic pushed a commit to chriseclectic/qiskit-aer that referenced this pull request Jan 25, 2021
* Add Python 3.9 support

This commit adds official python 3.9 support to qiskit-aer. This
includes CI, release notes, and package metadata.

Fixes Qiskit#1067

* Remove wheel job 3.9 skip

* Remove unused assertNoLogs method

The assertNoLogs methods was built using a private class from python's
stdlib unittest library. This should never have been done as it's
explicitly marked as private. Accordingly in Python 3.9 this private
class has been removed and no longer exists. It turns out this method
was not used anywhere in all of qiskit (the same identical code was
removed from terra as part of Qiskit/qiskit#5189 in
Qiskit/qiskit@ed5155b).
Since the implementation is not sound and nothing uses it this commit
just removes the class and method.

* Fix azure pipelines job matrix typo

Co-authored-by: Victor Villar <vvilpas@gmail.com>
chriseclectic pushed a commit to chriseclectic/qiskit-aer that referenced this pull request Jan 25, 2021
* Add Python 3.9 support

This commit adds official python 3.9 support to qiskit-aer. This
includes CI, release notes, and package metadata.

Fixes Qiskit#1067

* Remove wheel job 3.9 skip

* Remove unused assertNoLogs method

The assertNoLogs methods was built using a private class from python's
stdlib unittest library. This should never have been done as it's
explicitly marked as private. Accordingly in Python 3.9 this private
class has been removed and no longer exists. It turns out this method
was not used anywhere in all of qiskit (the same identical code was
removed from terra as part of Qiskit/qiskit#5189 in
Qiskit/qiskit@ed5155b).
Since the implementation is not sound and nothing uses it this commit
just removes the class and method.

* Fix azure pipelines job matrix typo

Co-authored-by: Victor Villar <vvilpas@gmail.com>
chriseclectic pushed a commit to chriseclectic/qiskit-aer that referenced this pull request Jan 25, 2021
* Add Python 3.9 support

This commit adds official python 3.9 support to qiskit-aer. This
includes CI, release notes, and package metadata.

Fixes Qiskit#1067

* Remove wheel job 3.9 skip

* Remove unused assertNoLogs method

The assertNoLogs methods was built using a private class from python's
stdlib unittest library. This should never have been done as it's
explicitly marked as private. Accordingly in Python 3.9 this private
class has been removed and no longer exists. It turns out this method
was not used anywhere in all of qiskit (the same identical code was
removed from terra as part of Qiskit/qiskit#5189 in
Qiskit/qiskit@ed5155b).
Since the implementation is not sound and nothing uses it this commit
just removes the class and method.

* Fix azure pipelines job matrix typo

Co-authored-by: Victor Villar <vvilpas@gmail.com>
chriseclectic pushed a commit to chriseclectic/qiskit-aer that referenced this pull request Jan 25, 2021
* Add Python 3.9 support

This commit adds official python 3.9 support to qiskit-aer. This
includes CI, release notes, and package metadata.

Fixes Qiskit#1067

* Remove wheel job 3.9 skip

* Remove unused assertNoLogs method

The assertNoLogs methods was built using a private class from python's
stdlib unittest library. This should never have been done as it's
explicitly marked as private. Accordingly in Python 3.9 this private
class has been removed and no longer exists. It turns out this method
was not used anywhere in all of qiskit (the same identical code was
removed from terra as part of Qiskit/qiskit#5189 in
Qiskit/qiskit@ed5155b).
Since the implementation is not sound and nothing uses it this commit
just removes the class and method.

* Fix azure pipelines job matrix typo

Co-authored-by: Victor Villar <vvilpas@gmail.com>
chriseclectic pushed a commit to chriseclectic/qiskit-aer that referenced this pull request Jan 25, 2021
* Add Python 3.9 support

This commit adds official python 3.9 support to qiskit-aer. This
includes CI, release notes, and package metadata.

Fixes Qiskit#1067

* Remove wheel job 3.9 skip

* Remove unused assertNoLogs method

The assertNoLogs methods was built using a private class from python's
stdlib unittest library. This should never have been done as it's
explicitly marked as private. Accordingly in Python 3.9 this private
class has been removed and no longer exists. It turns out this method
was not used anywhere in all of qiskit (the same identical code was
removed from terra as part of Qiskit/qiskit#5189 in
Qiskit/qiskit@ed5155b).
Since the implementation is not sound and nothing uses it this commit
just removes the class and method.

* Fix azure pipelines job matrix typo

Co-authored-by: Victor Villar <vvilpas@gmail.com>
chriseclectic pushed a commit to chriseclectic/qiskit-aer that referenced this pull request Jan 25, 2021
* Add Python 3.9 support

This commit adds official python 3.9 support to qiskit-aer. This
includes CI, release notes, and package metadata.

Fixes Qiskit#1067

* Remove wheel job 3.9 skip

* Remove unused assertNoLogs method

The assertNoLogs methods was built using a private class from python's
stdlib unittest library. This should never have been done as it's
explicitly marked as private. Accordingly in Python 3.9 this private
class has been removed and no longer exists. It turns out this method
was not used anywhere in all of qiskit (the same identical code was
removed from terra as part of Qiskit/qiskit#5189 in
Qiskit/qiskit@ed5155b).
Since the implementation is not sound and nothing uses it this commit
just removes the class and method.

* Fix azure pipelines job matrix typo

Co-authored-by: Victor Villar <vvilpas@gmail.com>
chriseclectic pushed a commit to chriseclectic/qiskit-aer that referenced this pull request Jan 25, 2021
* Add Python 3.9 support

This commit adds official python 3.9 support to qiskit-aer. This
includes CI, release notes, and package metadata.

Fixes Qiskit#1067

* Remove wheel job 3.9 skip

* Remove unused assertNoLogs method

The assertNoLogs methods was built using a private class from python's
stdlib unittest library. This should never have been done as it's
explicitly marked as private. Accordingly in Python 3.9 this private
class has been removed and no longer exists. It turns out this method
was not used anywhere in all of qiskit (the same identical code was
removed from terra as part of Qiskit/qiskit#5189 in
Qiskit/qiskit@ed5155b).
Since the implementation is not sound and nothing uses it this commit
just removes the class and method.

* Fix azure pipelines job matrix typo

Co-authored-by: Victor Villar <vvilpas@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stable-backport-potential The issue or PR might be minimal and/or import enough to backport to stable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Failed import on python version 3.9.0
3 participants