[NEW FEATURE] Local position in the validation catalogue for MCCD
Maintainer thường phản hồi trong vòng 1 ngày
@tobias-liaudat đang làm issue này rồi.
Từ ngày 26/1/2023.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
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)
- Ngôn ngữ chính
- Python
- Star
- 18
- Fork
- 14
- Merge trung bình
- 4 ngày 2 giờ
- Pull request đã merge (30 ngày)
- 26
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của CosmoStat/shapepipe
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
CosmoStat/shapepipe#929 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 38/100
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của CosmoStat/shapepipe
Issue tương tự
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
letsencrypt/cp-cps#353 ·
-
Marble Madness II is missingĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
PedestrianDynamics/pyFDS-Evac#394 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
DOI-USGS/pywatershed#421 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
python-pillow/Pillow#10087 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày