Skip to content
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.

Sign up for GitHub

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

Create pairwise_point_in_polygon to be used by pairwise GeoSeries #750

Merged
merged 19 commits into from
Nov 29, 2022
Merged

Create pairwise_point_in_polygon to be used by pairwise GeoSeries #750

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
121e146
Create new branch with pairwise_point_in_polygon cpp changes.
thomcom Oct 24, 2022
756282d
Clean up docs on cursory review.
thomcom Oct 24, 2022
f5f7777
Use T as TypeParam
thomcom Oct 24, 2022
a6f8f69
Add epsilon to colinearity test.
thomcom Oct 25, 2022
3a34016
Add EPSILON and refactor some tests.
thomcom Oct 25, 2022
e8af434
Remove stale comment.
thomcom Oct 25, 2022
70437da
Correct comments
thomcom Oct 25, 2022
c55ac72
Rename is_point_in_polygon.cuh
thomcom Nov 1, 2022
5624068
Keep only colinearity epsilon.
thomcom Nov 1, 2022
7550399
Change to grid/stride loop.
thomcom Nov 4, 2022
8786614
Merge branch 'branch-22.12' into feature/pairwise_point_in_polygon
thomcom Nov 28, 2022
be5a5c0
Fix bug with grid-stride loop and switch to for comparison.
thomcom Nov 28, 2022
f7bbf10
Update cpp/include/cuspatial/pairwise_point_in_polygon.hpp
thomcom Nov 28, 2022
b6d55b8
Update cpp/include/cuspatial/experimental/pairwise_point_in_polygon.cuh
thomcom Nov 28, 2022
d3d7533
Update cpp/include/cuspatial/experimental/pairwise_point_in_polygon.cuh
thomcom Nov 28, 2022
0edd2df
run-clang-format.py
thomcom Nov 28, 2022
1c02f3c
Improve grid stride loop computations.
thomcom Nov 29, 2022
89169ee
Update cpp/include/cuspatial/experimental/detail/is_point_in_polygon.cuh
thomcom Nov 29, 2022
e5e937f
Merge branch 'feature/pairwise_point_in_polygon' of github.com:thomco…
thomcom Nov 29, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update cpp/include/cuspatial/experimental/detail/is_point_in_polygon.cuh 
Co-authored-by: Michael Wang <isVoid@users.noreply.github.com>
  • Loading branch information
@thomcom @isVoid
thomcom and isVoid authored Nov 29, 2022
commit 89169ee56714e93bbe7875a7544c8e2028ff1c73
2 changes: 1 addition & 1 deletion cpp/include/cuspatial/experimental/detail/is_point_in_polygon.cuh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ __device__ inline bool is_point_in_polygon(Cart2d const& test_point,

// Points on the line segment are the same, so intersection is impossible.
// This is possible because we allow closed or unclosed polygons.
T zero = 0.0;
T constexpr zero = 0.0;
if (float_equal(run, zero) && float_equal(rise, zero)) continue;

T rise_to_point = test_point.y - a.y;
Expand Down