Map parameterization should be reverted
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- cpp
- Área
- computer-graphics
Línea de trabajo
Comienza revisando el commit c4aabc570103e72876e9889a20396e719f52bca4 y la ruta de renderd que aplica parameterize_function a los objetos de mapa. Compara ese comportamiento con las preocupaciones de las issues 2965 y 2553 de Mapnik. Se considera terminado cuando la mutación del mapa por renderizado se revierte o se reemplaza por un diseño acordado, sin realizar una copia innecesaria del mapa cuando no se usa la parametrización.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
/cc @jburgess777
Looking into ttps://github.com/mapnik/mapnik/issues/2965, I wondered if renderd was doing anything unsafe.
I noticed that https://github.com/openstreetmap/mod_tile/commit/c4aabc570103e72876e9889a20396e719f52bca4 added mutation of map objects per render. While this might work I think it should be reverted for at least these reasons:
- A copy of the
mapnik.Mapis made every time a tile is rendered even if noparameterize_functionis set. This is potentially very expensive. - I think parameterization would be much better done in XML before loading the map. Ideally maps should not be modified after being loaded.
- If per render changes to styling truly need to be made, then a much better design would be to leverage Mapnik 3.x variables: https://github.com/mapnik/mapnik/issues/2553
- Lenguaje dominante
- C++
- Estrellas
- 318
- Forks
- 200
- Merge medio
- 2 d 22 h
- PR fusionados (30 d)
- 22
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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.
Más de openstreetmap/mod_tile
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
openstreetmap/mod_tile#292 · 4 comentarios ·
Los mantenedores suelen responder en 2 días
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
openstreetmap/mod_tile#477 · 7 comentarios ·
Los mantenedores suelen responder en 2 días
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
openstreetmap/mod_tile#466 · 2 comentarios ·
Los mantenedores suelen responder en 2 días
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
openstreetmap/mod_tile#462 · 7 comentarios ·
Los mantenedores suelen responder en 2 días
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
openstreetmap/mod_tile#447 · 3 comentarios · 1 reacción ·
Los mantenedores suelen responder en 2 días
Todos los issues de openstreetmap/mod_tile
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
k2-fsa/sherpa-onnx#4100 ·
Los mantenedores suelen responder en 2 días
-
Eval bug: Vulkan: count_experts.comp zero_skipped_rows() loops over every id in every threadAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Mod: Part Design Topic: User Interface
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
A-tools P-medium S-ready-to-start task
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
sistemicorp/scripts#62 ·