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

Abierto Apto para principiantes
  • ¿Algo no está bien?
#789 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
80/100
Tipo de issue
Nueva funcionalidad
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
python
Área
data, tooling

Línea de trabajo

Empieza en tools/boundaries/gsak_to_opensak.py, donde _convert_countries, _convert_states y _convert_counties incrementan cada uno un contador missing cuando una ruta zip de origen es None. Reúne los códigos omitidos en una lista en cada pasada y luego imprímelos junto al resumen existente "X zip(s) missing" (o escríbelos en el manifiesto). Está terminado cuando una ejecución con un states/ch.zip faltante muestra ch en la salida.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.

Lenguaje dominante
Python
Estrellas
37
Forks
17
Merge medio
21 h 5 min
PR fusionados (30 d)
131

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de OpenSAK-Org/OpenSAK

Todos los issues de OpenSAK-Org/OpenSAK

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.