gsak_to_opensak.py silently skips missing source zips without logging which codes

Aperta Adatta ai principianti
  • Qualcosa non va?
#789 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
80/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
python
Ambito
data, tooling

Direzione di ricerca

Inizia da tools/boundaries/gsak_to_opensak.py, dove _convert_countries, _convert_states e _convert_counties incrementano ciascuno un contatore missing quando un percorso zip di origine è None. Raccogli i codici saltati in una lista in ogni passaggio, poi stampali accanto al riepilogo esistente "X zip(s) missing" (oppure scrivili nel manifest). È completato quando un'esecuzione con un states/ch.zip mancante riporta ch nell'output.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

area: geo-data good first issue improvement
Current State

tools/boundaries/gsak_to_opensak.py silently skips country/state/county packs when the corresponding source zip file isn't found on disk. The skip is only reflected in an aggregate count printed at the end (e.g. "2 zip(s) missing") — the specific country/pack codes that were skipped are never listed anywhere.

This came up in #788: Switzerland's state-level polygons were missing from the converted output, and the most likely cause was that states/ch.zip simply wasn't present in the gsak-dir at conversion time. But there was no way to know that from the tool's output — the gap only became visible months later when a user noticed blank state/county fields on Swiss caches.

Affected in _convert_countries, _convert_states, and _convert_counties — all three currently increment a missing counter on a None zip path without recording which code/pack triggered it.

Proposed Improvement

Collect skipped codes into a list per pass and print them (or write them to the manifest) alongside the existing "X missing" summary line.

Expected Benefits

When a source zip is missing, the tool should list the specific country/state/pack code(s) that were skipped (not just a total count), so gaps in source data are visible immediately after a conversion run rather than discovered later via a bug report.

Lingua principale
Python
Stelle
37
Fork
17
Merge medio
16h 7m
PR unite (30g)
128

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di OpenSAK-Org/OpenSAK

Tutte le issue di OpenSAK-Org/OpenSAK

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.