MapColonies / MapColonies/GpkgMerger
Mixed tile format doesn't work for S3 and FS
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- C#
- Estrellas
- 1
- Forks
- 2
- Merge medio
- 21 min
- PR fusionados (30 d)
- 1
Descripción
Describe the bug
When working with files, the merger uses file extension with no regard to existing file extension. This means that if a PNG file is merged with a new JPEG resulting in a full tile (JPEG) then it will be stored and both files would exist (for the same x,y,z).
To Reproduce
- Hold a tile with transparency in FS or S3
- Update with a tile that should result in a tile with no transparency
- New tile is written next to the existing tile
Expected behavior
Only the new calculated tile should exist.
Additional context
To combat this mapproxy uses .mixed extension, this way at the end the correct data would be placed. This is because the file extension isn't needed for discovering the tile format (can be understood from tile data).
If we implement the same strategy then we will be able to use mixed format with Mapproxy serving as well.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza localizando las rutas del merger que escriben tiles en FS y S3, y luego rastrea cómo se selecciona la extensión del archivo de salida cuando se combinan datos PNG y JPEG. Reproduce el escenario de transparencia a no transparencia del issue. Se considera terminado cuando el tile recién calculado reemplaza al tile existente en lugar de dejar ambas extensiones, y sigue siendo compatible con el serving de MapProxy.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- csharp
- Área
- backend, cloud
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 42/100