Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

Object-wise PSF-leakage correction zeros the same-object slope, not angular leakage; on v2 it raises τ0 by 52%

未关闭
#428 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
5/5
预计耗时
一周以上
新手友好度
8/100
Issue 类型
缺陷
描述清晰度
需要澄清
活跃度
活跃
技术栈
python

调研方向

The entry point is compute_PSF_leakage and the correction e_leak_corrected = e − α p, which fits α per SNR × size bin from the same-object galaxy–PSF slope. The payload does not name specific test files. Done would require a fit that targets angular leakage, validated on held-out stars, with per-bin α, errors and the global scalar persisted to the catalogue. This is design work and not a first-timer task.

由索引模型根据 Issue 内容生成。

描述

The object-wise PSF-leakage correction (compute_PSF_leakage → e_leak_corrected = e − α p) fits α from the same-object galaxy–PSF slope in SNR × size bins. Angular leakage is measured by ρ/τ at finite separation. These are different quantities, and on v2 they disagree in sign:

  • same-object slope (global scalar): −0.0030
  • the scalar that would zero first-bin τ0: +0.0058

So on v2, e_leak_corrected raises first-bin τ0 by 52% (5.25e-6 → 8.00e-6); on v1.4.6.3 it lowered τ0 by 59%. A perfect global same-object correction predicts the same 8.00e-6. The implementation itself checks out: the columns reconstruct to 4.6e-14, and the sign convention against stellar models is correct. Mechanism (verified): the same-object slope is a zero-lag mixture, α − β·Cov(p,δp)/Var(p) + η·Cov(p,q)/Var(p), where δp is the PSF ellipticity model error and q = p·δT_PSF/T_PSF the size-error term. The angular α, β, η, together with covariances from held-out stars, reproduce the measured slope: v2 −0.00310 predicted vs −0.00303 measured; v1.4.6.3 +0.0185 vs +0.0164. The size-error term is about −0.010 in both versions. v2's α is 2.5× smaller than v1.4.6.3's, so the sum turns negative. A per-object fit therefore can't isolate α, and a corrected v2 fiducial needs a fit that targets angular leakage.

Consequence: for v2, the _leak_corr columns are not a validated fiducial, so raw E is the reference.

Proposed:

  • make the target of compute_PSF_leakage explicit (object-wise vs angular on declared scales), and validate a correction on held-out stars before treating it as fiducial;
  • persist the fit diagnostics (per-bin α, errors, global scalar) with the catalogue;
  • label these columns as object-wise corrections in headers and docs.

Related: #382 (the trend-fit 1/σ⁴ weighting, negligible here at 5e-10), #413 (full a_ij / spin components).

Claude (Opus 5.5) on behalf of Cail

主要语言
Python
星标
2
派生
5
平均合并
1 天 1 小时
30 天内合并 PR
26

环境准备

  • 提供 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

CosmoStat/sp_validation 的其他 Issue

查看 CosmoStat/sp_validation 的全部 Issue

相似的 Issue

更多 Python Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。