Skip to content

Enable CI check with numpy-1.24.2 for #5991#6028

Closed
pavoljuhas wants to merge 5 commits intoquantumlib:masterfrom
pavoljuhas:core-np1.24-with-np1.24
Closed

Enable CI check with numpy-1.24.2 for #5991#6028
pavoljuhas wants to merge 5 commits intoquantumlib:masterfrom
pavoljuhas:core-np1.24-with-np1.24

Conversation

@pavoljuhas
Copy link
Copy Markdown
Collaborator

@pavoljuhas pavoljuhas commented Mar 9, 2023

DO NOT SUBMIT

Temporary CI checks related to compatibility with NumPy-1.24.

@CirqBot CirqBot added the size: M 50< lines changed <250 label Mar 9, 2023
@pavoljuhas pavoljuhas marked this pull request as ready for review March 9, 2023 01:32
@pavoljuhas pavoljuhas requested review from a team, cduck and vtomole as code owners March 9, 2023 01:32
@pavoljuhas pavoljuhas requested a review from verult March 9, 2023 01:32
@pavoljuhas pavoljuhas marked this pull request as draft March 9, 2023 01:32
Temporarily upgrade to numpy-1.24.2 to enable CI testing of quantumlib#5991.

Restrict CI-tests to Python 3.8 only which permits installation of
numpy-1.24.2 and numba.  Note numba is not yet compatible with
numpy-1.24 and its import raises SystemError exception
SystemError: initialization of _internal failed without raising an exception

Here we also add intentionally failing CI check to ensure this commit
is not accidentally merged to master.

Please revert this commit before merging the pull-request.
@pavoljuhas pavoljuhas force-pushed the core-np1.24-with-np1.24 branch from 7197575 to 90780ec Compare March 10, 2023 00:02
Also fail when attributes are used in eval expressions.

This prepares for upgrade to NumPy 1.24 which removes deprecated names.
@pavoljuhas
Copy link
Copy Markdown
Collaborator Author

withdrawn. CI-checks not feasible, numba is not yet available for numpy-1.24.

@pavoljuhas pavoljuhas closed this Mar 13, 2023
@pavoljuhas pavoljuhas deleted the core-np1.24-with-np1.24 branch March 13, 2023 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: M 50< lines changed <250

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants