Opened 3 years ago

Closed 3 years ago

#4944 closed defect (fixed)

Have ValidateTopology check side-labels of edges

Reported by: strk Owned by: strk
Priority: medium Milestone: PostGIS 3.2.0
Component: topology Version: master
Keywords: Cc:

Description

At this time ValidateTopology is NOT checking that the edges forming a ring (based on next_right_edge, next_left_edge) have consistent face labeling on the side they are walked though.

Chances are such invalidities will be reported by edge coverage check, but the edge coverege check is pretty expensive and might be skipped if we checked for side-labeling validity instead.

Change History (2)

comment:2 by Sandro Santilli <strk@…>, 3 years ago

Resolution: fixed
Status: newclosed

In 8b0ecf66/git:

Implement side labeling check in ValidateTopology

Closes #4944
Includes regress test catching new error (swapped side-labels)
Updates test expectances as we're now getting more errors.

Note: See TracTickets for help on using tickets.