Skip to content

Document NumPy 1 interaction with max simulation size#1015

Merged
mhucka merged 5 commits intoquantumlib:mainfrom
kris524:issue-863
Apr 6, 2026
Merged

Document NumPy 1 interaction with max simulation size#1015
mhucka merged 5 commits intoquantumlib:mainfrom
kris524:issue-863

Conversation

@kris524
Copy link
Copy Markdown
Contributor

@kris524 kris524 commented Feb 5, 2026

Fixes #863.

@github-actions github-actions bot added the size: XS <10 lines changed label Feb 5, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a helpful note to the documentation regarding issues with NumPy 1.26 in large simulations, recommending an upgrade to version 2.x. However, this creates a conflict with the requirements.txt file, which still permits the installation of NumPy 1.26. My review includes a comment to address this inconsistency to prevent potential issues for users.

Comment thread docs/choose_hw.md Outdated
Copy link
Copy Markdown
Collaborator

@mhucka mhucka left a comment

Choose a reason for hiding this comment

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

Thank you for this contribution. Please see the suggested edits in the review. If you are satisfied with the suggestions, you can use GitHub's feature for accepting edits directly, to make it easier to amend the PR.

Comment thread docs/choose_hw.md Outdated
Comment thread docs/choose_hw.md Outdated
kris524 and others added 3 commits April 5, 2026 11:12
Co-authored-by: Michael Hucka <mhucka@google.com>
Co-authored-by: Michael Hucka <mhucka@google.com>
@github-actions github-actions bot added the size: S 10< lines changed <50 label Apr 6, 2026
@mhucka mhucka enabled auto-merge April 6, 2026 02:13
@mhucka mhucka added this pull request to the merge queue Apr 6, 2026
Merged via the queue into quantumlib:main with commit d908952 Apr 6, 2026
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: S 10< lines changed <50 size: XS <10 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document NumPy 1 interaction with max simulation size

2 participants