Object-wise PSF-leakage correction zeros the same-object slope, not angular leakage; on v2 it raises τ0 by 52%
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 8/100
- Tipo de issue
- Error
- Claridad
- Necesita aclaración
- Estado de actividad
- Activo
- Stack tecnológico
- python
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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_leakageexplicit (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
- Lenguaje dominante
- Python
- Estrellas
- 2
- Forks
- 5
- Merge medio
- 1 d 1 h
- PR fusionados (30 d)
- 26
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de CosmoStat/sp_validation
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
CosmoStat/sp_validation#315 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 Medio día Aptitud para principiantes 65/100
CosmoStat/sp_validation#272 ·
Los mantenedores suelen responder en 1 día
-
Decide whether the catalogue cuts on ShapePipe's external_paired_fractionPosiblemente ocupada @cailmdaley la tomó hace 1 día. Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
CosmoStat/sp_validation#444 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
CosmoStat/sp_validation#443 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
CosmoStat/sp_validation#435 ·
Los mantenedores suelen responder en 1 día
Todos los issues de CosmoStat/sp_validation
Issues similares
-
Add `openai_safety_identifier`, the replacement for the `user` field that `openai_user` sendsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
pydantic/pydantic-ai#12091 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Enhance blog readabilityAbiertoCSS enhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
pychess/pychess-variants#2448 ·
Los mantenedores suelen responder en 1 día
-
Bump locked PyTensor so Apple clang 21 does not pass -ld64Posiblemente ocupada @drbenvincent la tomó hoy. Abiertobug dev ops
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 2 días
-
ai_p2 bug comp-datalake
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
ClickHouse/ClickHouse#128306 · 1 comentario ·
Los mantenedores suelen responder en 1 día