Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Remove the SIP to PV conversion

Đang mở
#612 3 bình luận 0 reaction 1 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@lbaumo đang làm issue này rồi.

Từ ngày 15/2/2023.

Đánh giá

Issue này chưa được đánh giá.

Mô tả

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.

Ngôn ngữ chính
Python
Star
18
Fork
14
Merge trung bình
3 ngày 19 giờ
Pull request đã merge (30 ngày)
25

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của CosmoStat/shapepipe

Tất cả issue của CosmoStat/shapepipe

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.