You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Gatti/DES weights pathological on image sims: N_eff ~1–5% of N, inflates σ(m) ~12× #227
Second finding from the end-to-end image-sim m-bias run (#225), distinct from the additive-subtraction bug: even with a paired (shape-noise-cancelling) estimator, the m-bias error stayed at ±1.5 instead of the expected ~±0.1. The w_des weights from compute_weights_gatti are degenerate on the sims — a handful of objects carry essentially all the weight.
(N_eff = (Σw)²/Σw²; a healthy shear weight sits at 50–90% of N.) σ(m) scales as 1/√N_eff, so this alone inflates the error ~8–12×, and the dominant few objects also skew the central value:
(All rows still show the m ≈ −1 central value — that's #226, a separate bug. The weight is the error story.) w_iv behaves normally, which is what isolates the problem to compute_weights_gatti — most likely the SNR/size binning going degenerate, or a variance term blowing up, on the sims' narrow magnitude/size range.
Why it can't be side-stepped
For the m-bias to be science-applicable it must be measured under the weight the science analysis actually uses — so w_des needs to be fixed at source, not swapped for w_iv (which serves as the diagnostic proving the estimator and pairing are sound). Possibly related to the broader ngmix-v2.0 weight-map work (inverse-variance weights, shapepipe#604 stream).
DES weights are very noisy for a small number of objects. In the (some) 20x20 grid of SNR and resolution, some pixels might have few or no galaxy for a small number of galaxies/tiles. Might have to run more tiles to see this behaviour go away?
Closing as resolved — no weights anywhere in the sim m-bias is now the standing configuration (w_col: null, global_R_weight: null in the image-sims branch config), with w_des retained only as a diagnostic knob.
Confirmation from the corrected single-tile baseline (tile 233.293, grid_1, paired unweighted estimator, and the true injected amplitude g = 0.025 — see the amplitude discussion around shapepipe#796): m1 = +0.019 ± 0.022, m2 = −0.015 ± 0.017, c1 = −0.0026 ± 0.0084, c2 = −0.0033 ± 0.0075. The σ(m) ≈ ±1.5 inflation this issue documents is gone; σ(m) ≈ 0.02 on a single tile with noise pairing (shapepipe#803).
The related estimator-flavor question (w_des appearing in the #796 m-bias snippet) is with Fabian on that thread; it doesn't affect this issue's verdict since the chain no longer uses weights anywhere.
Activity
martinkilbinger commentedon Jul 6, 2026
DES weights are very noisy for a small number of objects. In the (some) 20x20 grid of SNR and resolution, some pixels might have few or no galaxy for a small number of galaxies/tiles. Might have to run more tiles to see this behaviour go away?
cailmdaley commentedon Jul 7, 2026
@fabianhervaspeters says that we shouldn't be using weights at all for m-bias
feat(image-sims): w_col=None gives unit weights (no-weighting mode, #227
config(image-sims): unweighted global response for sim calibration (#227
Merge image_sims_val (Martin's parallel sim-validation stack) into the …
cailmdaley commentedon Jul 10, 2026
Closing as resolved — no weights anywhere in the sim m-bias is now the standing configuration (
w_col: null,global_R_weight: nullin the image-sims branch config), with w_des retained only as a diagnostic knob.Confirmation from the corrected single-tile baseline (tile 233.293, grid_1, paired unweighted estimator, and the true injected amplitude g = 0.025 — see the amplitude discussion around shapepipe#796): m1 = +0.019 ± 0.022, m2 = −0.015 ± 0.017, c1 = −0.0026 ± 0.0084, c2 = −0.0033 ± 0.0075. The σ(m) ≈ ±1.5 inflation this issue documents is gone; σ(m) ≈ 0.02 on a single tile with noise pairing (shapepipe#803).
The related estimator-flavor question (w_des appearing in the #796 m-bias snippet) is with Fabian on that thread; it doesn't affect this issue's verdict since the chain no longer uses weights anywhere.
— Fable on behalf of Cail
feat(image-sims): declare the sim mask config as an overlay on the da…
feat(image-sims): reconcile multi-weight-scheme m-bias with Phase A r…
Image-simulation m-bias: extracted core + snakemake orchestration (#225)
Record truth pass: image-simulation shear bias
Image-sim mask config keeps only the true reason for an unweighted R
Record truth pass: image-simulation shear bias
Image-sim mask config keeps only the true reason for an unweighted R
Record truth pass: image-simulation shear bias
Image-sim mask config keeps only the true reason for an unweighted R
Record truth pass: image-simulation shear bias
Image-sim mask config keeps only the true reason for an unweighted R