-
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
Add Header Only API pairwise_linestring_intersection
#852
Add Header Only API pairwise_linestring_intersection
#852
Conversation
…into feature/intersection_with_duplicates_pr
…into feature/merge_segment_pr
…into feature/merge_point_on_segment_pr
This reverts commit c6dc62c.
…into feature/intersection_with_duplicates_pr
…into feature/merge_point_on_segment_pr
…into feature/merge_segment_pr
Unfortunately, the original requirement for the output of intersection is unclear about what the user really expects here. I went afar to return the most cleaned-up output that the algorithm can return. (So does geopandas) It could be an early-optimization. It could be likely that it's not what the user wants. If you think the design is not meeting the need, feel free to file an issue to track. My question is, does this intersection function help your binary predicate use case? |
…diates_remove_if_test.cu Co-authored-by: H. Thomson Comer <thomcom@gmail.com>
…id/cuspatial into feature/header_only_intersection/pr
…into feature/header_only_intersection/pr
…into feature/header_only_intersection/pr
I'm sure this helps with implementing a great deal of binary predicates! I'm not exactly sure how, yet. I'm a little worried about a case of |
My questions are mostly general and for the purpose of understanding what's coming my way soon. :) I see @harrism already approved, I will approve if you want to get this merged before the week ends. |
…into feature/header_only_intersection/pr
@gpucibot merge |
Description
This PR adds header only API
pairwise_linestring_intersection
. Closes #765 .Depends on #813 #818 #819 #851
Checklist