reduce geojson maps precision and remove osm id

Author: davidgamez

functions-python/helpers/locations.py

@@ -231,3 +231,27 @@ def get_geopolygons_covers(stop_point: WKTElement, db_session: Session):
         ).all()
     )
     return geopolygons
+
+
+def round_geojson_coords(geometry, precision=5):
+    """Recursively round coordinates in a GeoJSON geometry dict to the given precision."""
+    if isinstance(geometry, dict):
+        if "coordinates" in geometry:
+            geometry = geometry.copy()
+            geometry["coordinates"] = round_coords(geometry["coordinates"], precision)
+        if "geometries" in geometry:  # GeometryCollection
+            geometry = geometry.copy()
+            geometry["geometries"] = [
+                round_geojson_coords(g, precision) for g in geometry["geometries"]
+            ]
+        return geometry
+    return geometry
+
+
+def round_coords(coords, precision):
+    if isinstance(coords, (list, tuple)):
+        if coords and isinstance(coords[0], (list, tuple)):
+            return [round_coords(c, precision) for c in coords]
+        else:
+            return [round(float(c), precision) for c in coords]
+    return coords

functions-python/reverse_geolocation/src/reverse_geolocation_processor.py

@@ -36,7 +36,7 @@
 )
 from shared.dataset_service.dataset_service_commons import Status
 
-from shared.helpers.locations import ReverseGeocodingStrategy
+from shared.helpers.locations import ReverseGeocodingStrategy, round_geojson_coords
 from shared.helpers.logger import get_logger
 from shared.helpers.runtime_metrics import track_metrics
 from shared.helpers.utils import (
@@ -169,9 +169,10 @@ def create_geojson_aggregate(
         "features": [
             {
                 "type": "Feature",
-                "geometry": mapping(geo_polygon_count[osm_id]["clipped_geometry"]),
+                "geometry": round_geojson_coords(
+                    mapping(geo_polygon_count[osm_id]["clipped_geometry"])
+                ),
                 "properties": {
-                    "osm_id": osm_id,
                     "country_code": geo_polygon_count[osm_id]["group"].iso_3166_1_code,
                     "subdivision_code": geo_polygon_count[osm_id][
                         "group"

functions-python/reverse_geolocation/tests/test_reverse_geolocation_processor.py

@@ -329,7 +329,6 @@ def test_create_geojson_aggregate(self, mock_getenv, mock_storage_client):
         self.assertEqual(geojson_data["type"], "FeatureCollection")
         self.assertEqual(len(geojson_data["features"]), 1)
         feature = geojson_data["features"][0]
-        self.assertEqual(feature["properties"]["osm_id"], str(geopolygon_1.osm_id))
         self.assertEqual(
             feature["properties"]["country_code"], geopolygon_1.iso_3166_1_code
         )

web-app/src/app/components/PopupTable.tsx

@@ -18,9 +18,6 @@ interface PopupTableProps {
 }
 
 const fieldDescriptions: Record<string, { description?: string }> = {
-  osm_id: {
-    description: 'OpenStreetMap ID of the geographic area.',
-  },
   stops_in_area: {
     description:
       'This is the number of stops in stops.txt that are located within this geographic area.',