Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Remove the SIP to PV conversion

Abierto
  • ¿Algo no está bien?
#612 3 comentarios 0 reacciones 1 asignado Ver en GitHub

Los mantenedores suelen responder en 1 día

@lbaumo ya está trabajando en esto.

Desde el 15/2/2023.

Evaluación

Este issue todavía no se ha evaluado.

Descripción

enhancement

In the split_exp package we are converting the WCS PV keywords to SIP to read the WCS with astropy, and this is not necessary. At the moment astropy cannot read the current WCS because the TYPE of the WCS is set to RA--TAN instead of RA--TPV. This was not a problem in the past because astropy allows inconsistency between the TYPE and the provided keywords, but now it is more strict about it. Also the TAN and TPV conventions are the same. I would propose to remove the conversion in the pipeline and instead update the TYPE in the header to avoid the dependence to an external package and changing the original WCS.

Lenguaje dominante
Python
Estrellas
18
Forks
14
Merge medio
1 d 3 h
PR fusionados (30 d)
19

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 CosmoStat/shapepipe

Todos los issues de CosmoStat/shapepipe

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.