Address PR feedback: simplify Hessen coordinate extraction

- Update type annotation to tuple[float, float] | tuple[None, None]
- Simplify README geolocation description (remove concrete numbers)
- Remove superfluous 'or None' from normalize() method

Author: tim

README.md

@@ -49,7 +49,7 @@ When available, we try to use the geolocations provided by the data publishers.
 | BB    | ✅ Yes                | WFS                                          |
 | HB    | ❌ No                 | -                                            |
 | HH    | ✅ Yes                | WFS                                          |
-| HE    | ⚠️  Partial (90.7%)    | Extracted from OSM on detail pages (1,863/2,054 schools). The 191 schools without coordinates include both schools with placeholder coordinates (-1.0, -1.0) that are filtered to null and schools with no map data at all. |
+| HE    | ⚠️  Partial (~90%)    | Extracted from OSM on detail pages. The schools without coordinates are schools with placeholder coordinates that are filtered out and schools with no map data at all. |
 | MV    | ❌ No                 | -                                            |
 | NI    | ❌ No                 | -                                            |
 | NW    | ✅ Yes                | Converted from EPSG:25832 in source CSV data |

jedeschule/spiders/hessen.py

@@ -53,7 +53,7 @@ def parse_list(self, response):
         for school in schools:
             yield scrapy.Request(school, callback=self.parse_details)
 
-    def _extract_coords_from_osm_url(self, url: str) -> tuple[float | None, float | None]:
+    def _extract_coords_from_osm_url(self, url: str) -> tuple[float, float] | tuple[None, None]:
         """Extract coordinates from OpenStreetMap iframe URL marker parameter"""
         qs = parse_qs(urlparse(url).query)
 
@@ -128,15 +128,15 @@ def parse_details(self, response):
     def normalize(item: Item) -> School:
         """Transform raw scraped data into standardized School model"""
         return School(
-            name=item.get("name") or None,
-            phone=item.get("telefon") or None,
-            fax=item.get("fax") or None,
-            website=item.get("homepage") or None,
-            address=item.get("straße") or None,
-            city=item.get("ort") or None,
-            zip=item.get("plz") or None,
-            school_type=item.get("schultyp") or None,
+            name=item.get("name"),
+            phone=item.get("telefon"),
+            fax=item.get("fax"),
+            website=item.get("homepage"),
+            address=item.get("straße"),
+            city=item.get("ort"),
+            zip=item.get("plz"),
+            school_type=item.get("schultyp"),
             id="HE-{}".format(item.get("id")),  # Prefix with state code
-            latitude=item.get("latitude") or None,
-            longitude=item.get("longitude") or None,
+            latitude=item.get("latitude"),
+            longitude=item.get("longitude"),
         )