-
Notifications
You must be signed in to change notification settings - Fork 6
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
Fix CI #60
Fix CI #60
Conversation
Signed-off-by: Vibhu Jawa <[email protected]>
Signed-off-by: Vibhu Jawa <[email protected]>
Signed-off-by: Vibhu Jawa <[email protected]>
Signed-off-by: Vibhu Jawa <[email protected]>
Signed-off-by: Vibhu Jawa <[email protected]>
Signed-off-by: Vibhu Jawa <[email protected]>
Signed-off-by: Vibhu Jawa <[email protected]>
Signed-off-by: Vibhu Jawa <[email protected]>
Signed-off-by: Vibhu Jawa <[email protected]>
Signed-off-by: Vibhu Jawa <[email protected]>
Signed-off-by: Vibhu Jawa <[email protected]>
Signed-off-by: Vibhu Jawa <[email protected]>
Signed-off-by: Vibhu Jawa <[email protected]>
CI seems to be passing again . CC: @marcromeyn / @edknv for quick eyes on this. |
@@ -29,7 +29,7 @@ def __init__(self): | |||
def concatenate(self, series_list, *, axis=None): | |||
return cudf.concat(series_list, axis=axis or 0) | |||
|
|||
np_backend_dispatch.register((cudf.Series, cudf.GenericIndex))(CudfBackend()) | |||
np_backend_dispatch.register((cudf.Series, cudf.Index))(CudfBackend()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unsure if this is the right fix. CC: @rjzamora
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this looks correct to me. My understanding is that all "real" index objects inherit from Index
now.
I suppose dask-cudf uses BaseIndex, but that dispatch was added a long time ago (and BaseIndex is probably unnecessary).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a ton for confirming this .
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Can also confirm this works fine with my NeMo Curator setup.
This PR enables GPU CI on this repo:
Tested locally using: