-
Notifications
You must be signed in to change notification settings - Fork 154
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
[REVIEW] from_geopandas
converts GeoPandas GeoSeries objects into cuspatial GeoArrow form.
#300
[REVIEW] from_geopandas
converts GeoPandas GeoSeries objects into cuspatial GeoArrow form.
#300
Conversation
…which are multi and which are not, yet, that'll take another consideration.
Please update the changelog in order to start CI tests. View the gpuCI docs here. |
Moving to 0.17. BTW, if you use a draft PR, there's no need for [WIP] / [REVIEW] anymore. |
Co-authored-by: Keith Kraus <kkraus@nvidia.com>
@kkraus14 I've resolved all of your comments! Please to confirm approval. |
🎉 |
@dillon-cullinan @ajschmidt8 looks like I need an ops approval, I don't think it is necessary. |
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!
from_geopandas
converts GeoPandas GeoSeries objects into cuspatial GeoArrow form.from_geopandas
converts GeoPandas GeoSeries objects into cuspatial GeoArrow form.
from_geopandas
converts GeoPandas GeoSeries objects into cuspatial GeoArrow form. from_geopandas
converts GeoPandas GeoSeries objects into cuspatial GeoArrow form.
@gpucibot merge |
This PR adds the
from_geopandas
feature, creating aGeoSeries
object backed by GPU memory and GeoArrow format as discussed with GeoPandas team members.This PR also provides
to_geopandas
, putting cuspatial GPU memory back into a GeoPandasGeoSeries
object.