uk_bin_collection 0.146.0__py3-none-any.whl → 0.146.2__py3-none-any.whl

uk_bin_collection/compare_lad_codes.py

@@ -0,0 +1,58 @@
+import json
+import geopandas as gpd
+
+def extract_lad_codes(input_json_path):
+    with open(input_json_path, "r") as f:
+        data = json.load(f)
+
+    lad_codes = set()
+    lad_code_to_council_input = {}
+
+    for council_key, council_info in data.items():
+        if isinstance(council_info, dict):
+            if "LAD24CD" in council_info:
+                code = council_info["LAD24CD"]
+                lad_codes.add(code)
+                lad_code_to_council_input[code] = council_key
+            if "supported_councils_LAD24CD" in council_info:
+                for code in council_info["supported_councils_LAD24CD"]:
+                    lad_codes.add(code)
+                    lad_code_to_council_input[code] = f"{council_key} (shared)"
+
+    return lad_codes, lad_code_to_council_input
+
+def compare_with_geojson(input_lad_codes, geojson_path):
+    gdf = gpd.read_file(geojson_path)
+    geojson_lad_codes = set(gdf["LAD24CD"].dropna().unique())
+
+    geojson_lad_map = {
+        row["LAD24CD"]: row["LAD24NM"]
+        for _, row in gdf.iterrows()
+        if "LAD24CD" in row and "LAD24NM" in row
+    }
+
+    missing_in_input = geojson_lad_codes - input_lad_codes
+    extra_in_input = input_lad_codes - geojson_lad_codes
+    matching = input_lad_codes & geojson_lad_codes
+
+    return matching, missing_in_input, extra_in_input, geojson_lad_map
+
+# --- Run the comparison ---
+input_json_path = "uk_bin_collection/tests/input.json"
+geojson_path = "uk_bin_collection/Local_Authority_Boundaries.geojson"
+
+input_lad_codes, input_name_map = extract_lad_codes(input_json_path)
+matching, missing, extra, geojson_name_map = compare_with_geojson(input_lad_codes, geojson_path)
+
+# --- Print results ---
+print(f"✅ Matching LAD24CDs ({len(matching)}):")
+for code in sorted(matching):
+    print(f"  {code} → input.json: {input_name_map.get(code)} | geojson: {geojson_name_map.get(code)}")
+
+print(f"\n🟡 LADs in GeoJSON but missing in input.json ({len(missing)}):")
+for code in sorted(missing):
+    print(f"  {code} → geojson: {geojson_name_map.get(code)}")
+
+print(f"\n🔴 LADs in input.json but not in GeoJSON ({len(extra)}):")
+for code in sorted(extra):
+    print(f"  {code} → input.json: {input_name_map.get(code)}")

uk_bin_collection/tests/input.json

@@ -1013,7 +1013,8 @@
             "N09000010",
             "S12000005",
             "S12000045",
-            "W06000020"
+            "W06000020",
+            "E07000122"
         ]
     },
     "GraveshamBoroughCouncil": {

uk_bin_collection/uk_bin_collection/councils/Hillingdon.py

@@ -35,41 +35,62 @@ DAYS_OF_WEEK = {
 def get_bank_holiday_changes(driver: WebDriver) -> Dict[str, str]:
     """Fetch and parse bank holiday collection changes from the council website."""
     bank_holiday_url = "https://www.hillingdon.gov.uk/bank-holiday-collections"
-    driver.get(bank_holiday_url)
+    changes: Dict[str, str] = {}
 
-    # Wait for page to load
-    wait = WebDriverWait(driver, 10)
-    wait.until(EC.presence_of_element_located((By.TAG_NAME, "table")))
+    try:
+        driver.get(bank_holiday_url)
 
-    # Parse the page
-    soup = BeautifulSoup(driver.page_source, features="html.parser")
-    changes: Dict[str, str] = {}
+        # Check if the page is a 404 or has an error
+        if "404" in driver.title or "Page not found" in driver.page_source:
+            print("Bank holiday page not found (404).")
+            return changes
 
-    # Find all tables with collection changes
-    tables = soup.find_all("table")
-    for table in tables:
-        # Check if this is a collection changes table
-        headers = [th.text.strip() for th in table.find_all("th")]
-        if "Normal collection day" in headers and "Revised collection day" in headers:
-            # Process each row
-            for row in table.find_all("tr")[1:]:  # Skip header row
-                cols = row.find_all("td")
-                if len(cols) >= 2:
-                    normal_date = cols[0].text.strip()
-                    revised_date = cols[1].text.strip()
-
-                    # Parse dates
-                    try:
-                        normal_date = parse(normal_date, fuzzy=True).strftime(
-                            "%d/%m/%Y"
-                        )
-                        revised_date = parse(revised_date, fuzzy=True).strftime(
-                            "%d/%m/%Y"
-                        )
-                        changes[normal_date] = revised_date
-                    except Exception as e:
-                        print(f"Error parsing dates: {e}")
-                        continue
+        # Wait for page to load
+        wait = WebDriverWait(driver, 10)
+        try:
+            wait.until(EC.presence_of_element_located((By.TAG_NAME, "table")))
+        except TimeoutException:
+            print("No tables found on the bank holiday page.")
+            return changes
+
+        # Parse the page
+        soup = BeautifulSoup(driver.page_source, features="html.parser")
+
+        # Find all tables with collection changes
+        tables = soup.find_all("table")
+        if not tables:
+            print("No relevant tables found on the bank holiday page.")
+            return changes
+
+        for table in tables:
+            # Check if this is a collection changes table
+            headers = [th.text.strip() for th in table.find_all("th")]
+            if (
+                "Normal collection day" in headers
+                and "Revised collection day" in headers
+            ):
+                # Process each row
+                for row in table.find_all("tr")[1:]:  # Skip header row
+                    cols = row.find_all("td")
+                    if len(cols) >= 2:
+                        normal_date = cols[0].text.strip()
+                        revised_date = cols[1].text.strip()
+
+                        # Parse dates
+                        try:
+                            normal_date = parse(normal_date, fuzzy=True).strftime(
+                                "%d/%m/%Y"
+                            )
+                            revised_date = parse(revised_date, fuzzy=True).strftime(
+                                "%d/%m/%Y"
+                            )
+                            changes[normal_date] = revised_date
+                        except Exception as e:
+                            print(f"Error parsing dates: {e}")
+                            continue
+
+    except Exception as e:
+        print(f"An error occurred while fetching bank holiday changes: {e}")
 
     return changes
 

{uk_bin_collection-0.146.0.dist-info → uk_bin_collection-0.146.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: uk_bin_collection
-Version: 0.146.0
+Version: 0.146.2
 Summary: Python Lib to collect UK Bin Data
 Author: Robert Bradley
 Author-email: robbrad182@gmail.com
@@ -9,6 +9,7 @@ Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
 Requires-Dist: bs4
+Requires-Dist: geopandas (>=1.0.1,<2.0.0)
 Requires-Dist: holidays
 Requires-Dist: icalevents (>=0.2.1,<0.3.0)
 Requires-Dist: lxml

{uk_bin_collection-0.146.0.dist-info → uk_bin_collection-0.146.2.dist-info}/RECORD

@@ -1,11 +1,12 @@
 uk_bin_collection/Local_Authority_Boundaries.geojson,sha256=_j-hUiL0--t2ewd_s29-j7_AKRlhagRMmOhXyco-B6I,1175922
 uk_bin_collection/README.rst,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
+uk_bin_collection/compare_lad_codes.py,sha256=BjXPzxYbbqyl2Pv9yPip0BVpyD3GOofwgWa-BQUecqM,2214
 uk_bin_collection/map.html,sha256=qKc4Lscv1eSIc0M3vDofzZf5w9_eslEYpz1-kK4tup0,4346
 uk_bin_collection/tests/check_selenium_url_in_input.json.py,sha256=lf-JT7vvaSfvgbrfOhzrhfSzJqL82WajlRqo1GqfcMM,7875
 uk_bin_collection/tests/council_feature_input_parity.py,sha256=DO6Mk4ImYgM5ZCZ-cutwz5RoYYWZRLYx2tr6zIs_9Rc,3843
 uk_bin_collection/tests/features/environment.py,sha256=VQZjJdJI_kZn08M0j5cUgvKT4k3iTw8icJge1DGOkoA,127
 uk_bin_collection/tests/features/validate_council_outputs.feature,sha256=SJK-Vc737hrf03tssxxbeg_JIvAH-ddB8f6gU1LTbuQ,251
-uk_bin_collection/tests/input.json,sha256=hJ_W7-yiUwGRA2V9WVQVXCV3UfaBFfbdaIbxm3c7hIM,133302
+uk_bin_collection/tests/input.json,sha256=-EVTLbgBwk879YMbAHkIYC7svXffzlL8xOa8Tab8M7g,133327
 uk_bin_collection/tests/output.schema,sha256=ZwKQBwYyTDEM4G2hJwfLUVM-5v1vKRvRK9W9SS1sd18,1086
 uk_bin_collection/tests/step_defs/step_helpers/file_handler.py,sha256=Ygzi4V0S1MIHqbdstUlIqtRIwnynvhu4UtpweJ6-5N8,1474
 uk_bin_collection/tests/step_defs/test_validate_council.py,sha256=VZ0a81sioJULD7syAYHjvK_-nT_Rd36tUyzPetSA0gk,3475
@@ -148,7 +149,7 @@ uk_bin_collection/uk_bin_collection/councils/HerefordshireCouncil.py,sha256=JpQh
 uk_bin_collection/uk_bin_collection/councils/HertsmereBoroughCouncil.py,sha256=ZbSsmqHStd2JtTMAq1Bhcvsj1BYp6ijELyOjZFX2GSw,6435
 uk_bin_collection/uk_bin_collection/councils/HighPeakCouncil.py,sha256=x7dfy8mdt2iGl8qJxHb-uBh4u0knmi9MJ6irOJw9WYA,4805
 uk_bin_collection/uk_bin_collection/councils/HighlandCouncil.py,sha256=GNxDU65QuZHV5va2IrKtcJ6TQoDdwmV03JvkVqOauP4,3291
-uk_bin_collection/uk_bin_collection/councils/Hillingdon.py,sha256=R1enDv5gjwCUT3HKgj8C87xWrwvrutAN6XLu5P7tef8,10532
+uk_bin_collection/uk_bin_collection/councils/Hillingdon.py,sha256=2OUp0iYO1YeZuTq0XRUalgoay5JRZgfHKKEwYzdMAU0,11291
 uk_bin_collection/uk_bin_collection/councils/HinckleyandBosworthBoroughCouncil.py,sha256=51vXTKrstfJhb7cLCcrsvA9qKCsptyNMZvy7ML9DasM,2344
 uk_bin_collection/uk_bin_collection/councils/HullCityCouncil.py,sha256=UHcesBoctFVcXDYuwfag43KbcJcopkEDzJ-54NxtK0Q,1851
 uk_bin_collection/uk_bin_collection/councils/HuntingdonDistrictCouncil.py,sha256=dGyhhG6HRjQ2SPeiRwUPTGlk9dPIslagV2k0GjEOn1s,1587
@@ -332,8 +333,8 @@ uk_bin_collection/uk_bin_collection/councils/YorkCouncil.py,sha256=I2kBYMlsD4bId
 uk_bin_collection/uk_bin_collection/councils/council_class_template/councilclasstemplate.py,sha256=QD4v4xpsEE0QheR_fGaNOIRMc2FatcUfKkkhAhseyVU,1159
 uk_bin_collection/uk_bin_collection/create_new_council.py,sha256=m-IhmWmeWQlFsTZC4OxuFvtw5ZtB8EAJHxJTH4O59lQ,1536
 uk_bin_collection/uk_bin_collection/get_bin_data.py,sha256=YvmHfZqanwrJ8ToGch34x-L-7yPe31nB_x77_Mgl_vo,4545
-uk_bin_collection-0.146.0.dist-info/LICENSE,sha256=vABBUOzcrgfaTKpzeo-si9YVEun6juDkndqA8RKdKGs,1071
-uk_bin_collection-0.146.0.dist-info/METADATA,sha256=HDx_ThJ-YHiak_qHaFdhBG4hOT-xeil2DR5LO4V_kAA,19858
-uk_bin_collection-0.146.0.dist-info/WHEEL,sha256=Nq82e9rUAnEjt98J6MlVmMCZb-t9cYE2Ir1kpBmnWfs,88
-uk_bin_collection-0.146.0.dist-info/entry_points.txt,sha256=36WCSGMWSc916S3Hi1ZkazzDKHaJ6CD-4fCEFm5MIao,90
-uk_bin_collection-0.146.0.dist-info/RECORD,,
+uk_bin_collection-0.146.2.dist-info/LICENSE,sha256=vABBUOzcrgfaTKpzeo-si9YVEun6juDkndqA8RKdKGs,1071
+uk_bin_collection-0.146.2.dist-info/METADATA,sha256=1F2N9JtI_FSX2IALiojB8bO-kjCn1N2tNyAAnKGuv6s,19900
+uk_bin_collection-0.146.2.dist-info/WHEEL,sha256=Nq82e9rUAnEjt98J6MlVmMCZb-t9cYE2Ir1kpBmnWfs,88
+uk_bin_collection-0.146.2.dist-info/entry_points.txt,sha256=36WCSGMWSc916S3Hi1ZkazzDKHaJ6CD-4fCEFm5MIao,90
+uk_bin_collection-0.146.2.dist-info/RECORD,,