[NEW FEATURE] Local position in the validation catalogue for MCCD
Los mantenedores suelen responder en 1 día
@tobias-liaudat ya está trabajando en esto.
Desde el 26/1/2023.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
Hello,
I have noticed that in the MCCD catalogue the x and y positions are given on the global grid that MCCD is using for the multi-CCD approach. Due to this, the MCCD package is required to read the catalogue and transform back the positions to a local reference mining x and y on each CCDs. I think it could be nice to have the local positions directly in the catalogue to avoid the call to MCCD. That would simplify the validation a lot because the catalogue will follow the same format as PSFEx and we could have only one validation function instead of two. I think that could also simplify the comprehension of the catalogue by external users.
I see two ways to handle this:
- This can be done in the
merge_starcatmodule but I found that a bit sub-optimal because we will have one function for MCCD catalogues and one for PSFEx (like it is right now). - Do it in MCCD when it does the interpolation. Like that we could also simplify the
merge_starcatmodule. What I would propose is just to add one column for the LOCAL_POS along side the GLOBAL_POS so the user can choose which one to. It will not break the pipeline as it is and give time to update the different modules that are impacted.
One solution is only in ShapePipe the other is in MCCD. I have a preference for the second one. Let me know what you think 🙂
(Maybe this issue should be moved to the MCCD repo, let me know)
- Lenguaje dominante
- Python
- Estrellas
- 18
- Forks
- 14
- Merge medio
- 1 d 3 h
- PR fusionados (30 d)
- 19
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
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 CosmoStat/shapepipe
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Los mantenedores suelen responder en 1 día
Todos los issues de CosmoStat/shapepipe
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
qgis/QGIS-Plugins-Website#459 ·
-
bug severity:medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
bot-found bug priority: P3
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
madenvel/KalinkaPlayer#179 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
ls1intum/edutelligence#1098 ·
Los mantenedores suelen responder en 1 día