Skip to content

v.voronoi: fix skeleton creation - #7513

Merged
echoix merged 1 commit into
OSGeo:mainfrom
metzm:v.voronoi_skeleton
Jun 7, 2026
Merged

echoix merged 1 commit into
OSGeo:mainfrom
metzm:v.voronoi_skeleton

Conversation

@metzm

@metzm metzm commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

With recent changes to vector libs, point-in-polygon calculations became more strict. A point exactly on a boundary can not be assigned to one specific area. For skeleton creation, points on boundaries are actually desired. This PR adjusts the test in v.voronoi when snapping skeleton nodes to an area's boundary. This affects also GRASS 8.5, thus backport is needed.

@metzm metzm added this to the 8.6.0 milestone Jun 6, 2026
@metzm metzm added bug Something isn't working vector Related to vector data processing C Related code is in C backport to 8.5 PR needs to be backported to release branch 8.5 labels Jun 6, 2026
@github-actions github-actions Bot added the module label Jun 6, 2026
@metzm

metzm commented Jun 6, 2026

Copy link
Copy Markdown
Contributor Author

The test r.sim.water/testsuite/test_r_sim_water.py test_nodxdy fails, this is a known issue unrelated to this PR. Triggering the test again in the hope it will succeed is a waste of resources. The r.sim.water test needs to be deactivated until it is fixed..

@echoix

echoix commented Jun 7, 2026

Copy link
Copy Markdown
Member

The test r.sim.water/testsuite/test_r_sim_water.py test_nodxdy fails, this is a known issue unrelated to this PR. Triggering the test again in the hope it will succeed is a waste of resources. The r.sim.water test needs to be deactivated until it is fixed..

There’s work going on that.

@echoix
echoix merged commit 0f6f0a5 into OSGeo:main Jun 7, 2026
26 of 27 checks passed
@metzm
metzm deleted the v.voronoi_skeleton branch June 11, 2026 20:14
metzm added a commit that referenced this pull request Jun 11, 2026
With recent changes to vector libs, point-in-polygon calculations became more strict. A point exactly on a boundary can not be assigned to one specific area. For skeleton creation, points on boundaries are actually desired. This PR adjusts the test in `v.voronoi` when snapping skeleton nodes to an area's boundary.
ninsbl pushed a commit to ninsbl/grass that referenced this pull request Jul 7, 2026
With recent changes to vector libs, point-in-polygon calculations became more strict. A point exactly on a boundary can not be assigned to one specific area. For skeleton creation, points on boundaries are actually desired. This PR adjusts the test in `v.voronoi` when snapping skeleton nodes to an area's boundary.
@neteler neteler removed the backport to 8.5 PR needs to be backported to release branch 8.5 label Jul 13, 2026
@neteler neteler modified the milestones: 8.6.0, 8.5.1 Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working C Related code is in C module vector Related to vector data processing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants