ngmix PSF fitter: drop the galaxy prior, adopt a metadetect-style dedicated PSF fit
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
检查 ngmix 模块当前的共享 fitter 设置，然后将其与 fitting.py 中 metadetect 的 get_gauss_psf_runner 进行比较。完成的标准是：PSF 使用自己的 gauss fitter，不使用 galaxy prior，并使用基于 moments 的 guesser，同时 galaxy fitter 保持不变。
由索引模型根据 Issue 内容生成。
描述
Longer-term follow-up to #749. The ngmix module fits the PSF with model="gauss" but using the galaxy shape prior, so the PSF fit is partly prior-driven (the #749 concern). The acute problem is mitigated by giving the PSF fit a proper weight map, and the prior — in Axel's words — "is not ideal but not completely wrong," so this is not a merge blocker. The clean fix is a dedicated PSF fitter that doesn't borrow the galaxy prior at all.
Axel's suggestion (#749 comment):
The prior is not ideal to fit the PSF but it is not completely wrong either. I would put that as a lower priority given that it still works and simplify things. Otherwise you can define a better PSF fitter like they do in metadetect here.
What metadetect does there (get_gauss_psf_runner): a gauss Fitter with no prior, paired with a SimplePSFGuesser(guess_from_moms=True) that initializes from the PSF's own adaptive moments — distinct from the galaxy fitter (get_gauss_obj_runner), which is the one that carries the prior. The PSF and galaxy fits are deliberately separate, and the PSF fit never sees the galaxy prior.
Improvement: give the ngmix module its own PSF fitter on the same pattern — model="gauss", no galaxy prior, a moments-based guesser — replacing the current shared-prior setup.
Priority: low (post-ngmix-v2.0; not blocking the merge).
— Claude on behalf of Cail
- 主要语言
- Python
- 星标
- 18
- 派生
- 14
- 平均合并
- 2 天 13 小时
- 30 天内合并 PR
- 35
环境准备
- 提供 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
CosmoStat/shapepipe 的其他 Issue
-
难度 1/5 1 小时以内 新手友好度 85/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 75/100
维护者通常 1 天内回复
-
难度 4/5 3-5 天 新手友好度 45/100
维护者通常 1 天内回复
-
Workflow: sp run refreshes the code snapshot before the lock; re-clean erases exposure tombstones; missing run-dir undetected可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭
难度 4/5 3-5 天 新手友好度 45/100
维护者通常 1 天内回复
-
难度 5/5 一周以上 新手友好度 18/100
CosmoStat/shapepipe#956 · 2 条评论 ·
维护者通常 1 天内回复
查看 CosmoStat/shapepipe 的全部 Issue
相似的 Issue
-
docs pydanty:is-working
难度 2/5 1-3 小时 新手友好度 68/100
pydantic/pydantic-ai#10113 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 77/100
GreedyBear-Project/GreedyBear#1727 ·
维护者通常 2 天内回复
-
难度 2/5 1-3 小时 新手友好度 84/100
ARM-software/CMSIS-DSP#354 ·
-
难度 2/5 1-3 小时 新手友好度 68/100
kiat/codeAssist#391 ·
-
needs triage
难度 2/5 1-3 小时 新手友好度 83/100
darnitdevorg/darnit#587 ·
维护者通常 1 天内回复