search

rapidsai / cuspatial

CUDA-accelerated GIS and spatiotemporal algorithms
https://docs.rapids.ai/api/cuspatial/stable/
Apache License 2.0
578 stars 145 forks source link

[FEA]: add cuspatial.GeoSeries.within to benchmarks #1354

Open raybellwaves opened 4 months ago

raybellwaves commented 4 months ago

Is this a new feature, an improvement, or a change to existing functionality?

Improvement

How would you describe the priority of this feature request

Low (would be nice)

Please provide a clear description of problem you would like to solve.

I got a nice speed up doing cgdf["geometry"].within(cgeom) compared to geopandas. I took a quick look at the benchmarks in https://github.com/rapidsai/cuspatial/blob/branch-24.04/python/cuspatial/benchmarks/api/bench_api.py but didn't find it immediately. It may behave similar to point_in_polygon but my cgeom is a MULTIPOLYGON

Describe any alternatives you have considered

No response

Additional context

It may be worth having a benchmark notebook in https://github.com/rapidsai/cuspatial/tree/branch-24.04/docs/source/user_guide similar to https://github.com/rapidsai/cudf/blob/branch-24.04/docs/cudf/source/user_guide/performance-comparisons/performance-comparisons.ipynb

GPUtester commented 4 months ago

Hi @raybellwaves!

Thanks for submitting this issue - our team has been notified and we'll get back to you as soon as we can! In the mean time, feel free to add any relevant information to this issue.