Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

[NEW FEATURE] Local position in the validation catalogue for MCCD

オープン
#610 コメント 0 件 リアクション 1 件 担当者 1 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

@tobias-liaudat がすでに取り組んでいます。

2023年1月26日 から。

評価

この issue はまだ評価されていません。

説明

enhancement

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:

  1. This can be done in the merge_starcat module 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).
  2. Do it in MCCD when it does the interpolation. Like that we could also simplify the merge_starcat module. 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)

主要言語
Python
スター
18
フォーク
14
平均マージ
1日 3時間
マージ済み PR(30日)
19

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

CosmoStat/shapepipe のほかの issue

CosmoStat/shapepipe の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。