Replace `cudf.core` imports with public APIs

rapidsai / cuspatial

CUDA-accelerated GIS and spatiotemporal algorithms

https://docs.rapids.ai/api/cuspatial/stable/

Apache License 2.0

604 stars 152 forks source link

Replace `cudf.core` imports with public APIs #1356

Closed mroeschke closed 7 months ago

mroeschke commented 7 months ago

Description

This PR replaces cudf.core imports of Series and DataFrame with their public API equivalents.

Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.

isVoid commented 7 months ago

The PR title seems to suggest that it's replacing all internal API usage in cuspatial with public APIs - which is a bit misleading IMO. How about be more specific: "replace cudf.core import with public APIs"?

bdice commented 7 months ago

I updated the description to reflect the changes. We can go ahead and merge this now.

bdice commented 7 months ago

/merge