Skip to content

Conversation

mtreinish
Copy link
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 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
Collaborator

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

  • @Qiskit/terra-core

@coveralls
Copy link

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 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