-
Notifications
You must be signed in to change notification settings - Fork 18
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
Test suite failure on 32bit architectures #133
Comments
Thanks for the bug report @avalentino can you change Lines 229 to 230 in e405517
If that works, a pull request with that change would be very welcome |
Dear @dcherian, unfortunately it still does not work, but th error seems to be a little bit different now:
|
What is the dtype of I've consistently used |
Yes, this is the way to go: diff --git a/tests/test_core.py b/tests/test_core.py
index 9c0bd6a..c70ae83 100644
--- a/tests/test_core.py
+++ b/tests/test_core.py
@@ -504,7 +504,7 @@ def test_groupby_reduce_nans(chunks, axis, groups, expected_shape, engine):
fill_value=0,
engine=engine,
)
- assert_equal(result, np.zeros(expected_shape, dtype=np.int64))
+ assert_equal(result, np.zeros(expected_shape, dtype=np.intp))
# now when subsets are NaN
# labels = np.array([0, 0, 1, 1, 1], dtype=float) |
OK can you send in a PR so you get credit for it? |
Done: #136. |
The text was updated successfully, but these errors were encountered: