Skip to content

Avoid np._bool #5368

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

Merged
merged 3 commits into from
May 16, 2022
Merged

Avoid np._bool #5368

merged 3 commits into from
May 16, 2022

Conversation

dabacon
Copy link
Collaborator

@dabacon dabacon commented May 16, 2022

np.bool is being deprecated by numpy for just bool, and while one can use np._bool, I think in many cases it is better to just use the native boolean.

This was tested against current and next mypy.

@dabacon dabacon requested review from a team, vtomole and cduck as code owners May 16, 2022 17:17
@dabacon dabacon requested a review from maffoo May 16, 2022 17:17
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@CirqBot CirqBot added the size: S 10< lines changed <50 label May 16, 2022
@dabacon dabacon added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label May 16, 2022
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label May 16, 2022
@CirqBot CirqBot merged commit b1a5a39 into quantumlib:master May 16, 2022
@CirqBot CirqBot removed automerge Tells CirqBot to sync and merge this PR. (If it's running.) front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. labels May 16, 2022
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
`np.bool` is being deprecated by numpy for just `bool`, and while one can use `np._bool`, I think in many cases it is better to just use the native boolean.  

This was tested against current and next mypy.
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
`np.bool` is being deprecated by numpy for just `bool`, and while one can use `np._bool`, I think in many cases it is better to just use the native boolean.  

This was tested against current and next mypy.
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants