Skip to content

Revert "Relax CI constraint on numpy<1.25 (#11020)"#11232

Merged
jakelishman merged 1 commit into
Qiskit:mainfrom
mtreinish:revert-ci-cap-numpy
Nov 11, 2023
Merged

Revert "Relax CI constraint on numpy<1.25 (#11020)"#11232
jakelishman merged 1 commit into
Qiskit:mainfrom
mtreinish:revert-ci-cap-numpy

Conversation

@mtreinish
Copy link
Copy Markdown
Member

Summary

Removing the restriction on the numpy version has introduced some non-determinism to some of the unitary synthesis tests that is causing a flaky failure in CI. This commit reverts the version cap removal to unblock CI. In parallel we should debug the source of the instability causing the test to fail so we can run CI using the latest version of numpy.

Details and comments

This reverts commit fb8a69d.

Removing the restriction on the numpy version has introduced some
non-determinism to some of the unitary synthesis tests that is causing a
flaky failure in CI. This commit reverts the version cap removal to
unblock CI. In parallel we should debug the source of the instability
causing the test to fail so we can run CI using the latest version of
numpy.

This reverts commit fb8a69d.
@mtreinish mtreinish added type: qa Issues and PRs that relate to testing and code quality Changelog: None Do not include in the GitHub Release changelog. labels Nov 10, 2023
@mtreinish mtreinish requested a review from a team as a code owner November 10, 2023 22:34
@qiskit-bot
Copy link
Copy Markdown
Collaborator

One or more of the the following people are requested to review this:

  • @Qiskit/terra-core

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 6830264563

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 9 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.01%) to 86.952%

Files with Coverage Reduction New Missed Lines %
crates/qasm2/src/lex.rs 3 93.18%
crates/qasm2/src/parse.rs 6 97.6%
Totals Coverage Status
Change from base Build 6828208042: 0.01%
Covered Lines: 74353
Relevant Lines: 85510

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changelog: None Do not include in the GitHub Release changelog. type: qa Issues and PRs that relate to testing and code quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants