[postgis-tickets] [SCM] PostGIS branch main updated. 3.1.0rc1-291-g61638f9

[git at osgeo.org]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "PostGIS".

The branch, main has been updated
       via  61638f97778d8a75bca8cba9117244f129179746 (commit)
      from  c40f25324c1e7d54f3ba78b20b662b142f09b235 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 61638f97778d8a75bca8cba9117244f129179746
Author: Sandro Santilli <strk at kbt.io>
Date:   Thu Jul 8 18:27:27 2021 +0200

    Make ValidateTopology mixed face labeling errors more predictable
    
    Will start rings from edge with lowest signed id

diff --git a/topology/sql/manage/ValidateTopology.sql.in b/topology/sql/manage/ValidateTopology.sql.in
index ceca2ec..9c813e8 100644
--- a/topology/sql/manage/ValidateTopology.sql.in
+++ b/topology/sql/manage/ValidateTopology.sql.in
@@ -247,7 +247,7 @@ BEGIN
   SELECT -edge_id FROM edges_to_check
   ;
 
-  CREATE INDEX ON side_label_check_edge (edge_id); -- DROP ?
+  CREATE INDEX ON side_label_check_edge (edge_id);
 
   RAISE DEBUG 'Checking edge side faces labeling';
 
@@ -258,6 +258,7 @@ BEGIN
     -- Fetch next unvisited edge
     SELECT edge_id
     FROM pg_temp.side_label_check_edge
+    ORDER BY edge_id
     LIMIT 1
     INTO nextEdge;
 
diff --git a/topology/test/regress/validatetopology_expected b/topology/test/regress/validatetopology_expected
index f21a9dd..340796f 100644
--- a/topology/test/regress/validatetopology_expected
+++ b/topology/test/regress/validatetopology_expected
@@ -11,19 +11,19 @@
 #4830.1|---||
 #4830.1|edge not covered by both its side faces|4|
 #4830.1|edge not covered by both its side faces|6|
-#4830.1|mixed face labeling in ring|3|
+#4830.1|mixed face labeling in ring|-6|
 #4830.2|---||
 #4830.2|edge not covered by both its side faces|4|
 #4830.2|edge not covered by both its side faces|6|
-#4830.2|mixed face labeling in ring|3|
+#4830.2|mixed face labeling in ring|-6|
 #4830.3|---||
 #4830.3|edge not covered by both its side faces|7|
 #4830.3|edge not covered by both its side faces|8|
-#4830.3|mixed face labeling in ring|-7|
+#4830.3|mixed face labeling in ring|-8|
 #4830.4|---||
 #4830.4|edge not covered by both its side faces|7|
 #4830.4|edge not covered by both its side faces|8|
-#4830.4|mixed face labeling in ring|-7|
+#4830.4|mixed face labeling in ring|-8|
 #4830.5|---||
 #4830.5|edge covered by some face has universal face on both sides|9|
 #3042|---||
@@ -36,13 +36,13 @@
 #3042|invalid next_right_edge|19|-10
 #3042|invalid next_right_edge|25|25
 #3042|mixed face labeling in ring|-22|
+#3042|mixed face labeling in ring|-21|
 #3042|mixed face labeling in ring|-20|
 #3042|mixed face labeling in ring|-10|
 #3042|mixed face labeling in ring|-9|
-#3042|mixed face labeling in ring|-7|
+#3042|mixed face labeling in ring|-6|
 #3042|mixed face labeling in ring|-3|
-#3042|mixed face labeling in ring|13|
 #3042|mixed face labeling in ring|19|
 #4944|---||
+#4944|mixed face labeling in ring|-21|
 #4944|mixed face labeling in ring|-19|
-#4944|mixed face labeling in ring|19|

-----------------------------------------------------------------------

Summary of changes:
 topology/sql/manage/ValidateTopology.sql.in     |  3 ++-
 topology/test/regress/validatetopology_expected | 14 +++++++-------
 2 files changed, 9 insertions(+), 8 deletions(-)


hooks/post-receive
-- 
PostGIS