jsontogeojson: add any_spans_with_geometry and any_nodes_with_geometr…

…y to meta Helpful for Open-Telecoms-Data/cove-ofds#6
Open-Telecoms-Data · Nov 9, 2022 · c5d3816 · c5d3816
1 parent ae77123
commit c5d3816
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -14,6 +14,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
   - Add more fields to output object
 - GeoJSON to JSON:
   - meta information includes inconsistent ids on organisations and phases
+- JSON to GeoJSON:
+  - add any_spans_with_geometry and any_nodes_with_geometry to meta
 - Python Validate:
   - add more information to errors
 

diff --git a/libcoveofds/geojson.py b/libcoveofds/geojson.py
@@ -69,6 +69,13 @@ def get_meta_json(self) -> dict:
                 out["spans_output_field_coverage"][key] = {"count": 1}
             else:
                 out["spans_output_field_coverage"][key]["count"] += 1
+        # Any geometries?
+        out["any_spans_with_geometry"] = bool(
+            [True for s in self._spans_geojson_features if s.get("geometry")]
+        )
+        out["any_nodes_with_geometry"] = bool(
+            [True for n in self._nodes_geojson_features if n.get("geometry")]
+        )
         # return
         return out
 

diff --git a/tests/fixtures/json_to_geojson/basic_1.expected.meta.json b/tests/fixtures/json_to_geojson/basic_1.expected.meta.json
@@ -116,5 +116,7 @@
         "/features/properties/network/name": {
             "count": 1
         }
-    }
+    },
+    "any_spans_with_geometry": true,
+    "any_nodes_with_geometry": true
 }
diff --git a/tests/fixtures/json_to_geojson/no_geometry_1.expected.meta.json b/tests/fixtures/json_to_geojson/no_geometry_1.expected.meta.json
@@ -86,5 +86,7 @@
         "/features/properties/network/name": {
             "count": 1
         }
-    }
+    },
+    "any_spans_with_geometry": false,
+    "any_nodes_with_geometry": false
 }
diff --git a/tests/fixtures/json_to_geojson/organisations_1.expected.meta.json b/tests/fixtures/json_to_geojson/organisations_1.expected.meta.json
@@ -176,5 +176,7 @@
         "/features/properties/network/name": {
             "count": 1
         }
-    }
+    },
+    "any_spans_with_geometry": true,
+    "any_nodes_with_geometry": true
 }
diff --git a/tests/fixtures/json_to_geojson/phases_1.expected.meta.json b/tests/fixtures/json_to_geojson/phases_1.expected.meta.json
@@ -200,5 +200,7 @@
         "/features/properties/network/contracts/relatedPhases/description": {
             "count": 1
         }
-    }
+    },
+    "any_spans_with_geometry": true,
+    "any_nodes_with_geometry": true
 }