-
Notifications
You must be signed in to change notification settings - Fork 85
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
support bool as dtype #691
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #691 +/- ##
==========================================
+ Coverage 74.37% 74.57% +0.19%
==========================================
Files 58 58
Lines 6774 6776 +2
Branches 1397 1398 +1
==========================================
+ Hits 5038 5053 +15
+ Misses 1338 1332 -6
+ Partials 398 391 -7
Continue to review full report at Codecov.
|
@ajtritt is this ready to go? Are there other things we need to change? |
@bendichter yes... someone just needs to review it ;) |
I'd like to add a test for this |
Motivation
Fix #658
Checklist
flake8
from the source directory.#XXX
notation whereXXX
is the issue number ?