DC2018: the C29 caveat was wrong about where the convention residual lands - #304
Merged
Merged
Conversation
ab194/av_true -- the first run with the colour-anchored prior (av = 9.01 +/-
0.3, ptde, Rhat 1.00, ESS 23-25k, 78/78 chains in the good region) -- says
the caveat this script prints was wrong, so it is rewritten rather than
quietly left in place.
IT CLAIMED theta_star IS BIASED LOW BY 10**(-0.2*grey), i.e. 0.766x on event
194, read straight off the challenge's CSB relation. MEASURED: 0.983x.
The arithmetic is right about the CSB relation and wrong about the target --
theta_star is PROTECTED, because the light curve pins rho * theta_E
independently of the SED. (Cross-checked: rho = theta_star / theta_E
reproduces both the truth and the fitted rho from the scored table.)
WHERE IT ACTUALLY LANDS. R_source and D_source are each ~2.45x low but
slide IN LOCKSTEP -- 0.403x and 0.410x -- so their ratio, which IS
theta_star, survives while each is individually -5 sigma. The flux the grey
term over-predicts is absorbed by a source made too COOL (teffsed 3040 K
here); the distance then follows the luminosity inward.
AND THE PRIOR FIX IS A REAL WIN THE OLD TEXT BURIED:
theta_star vs truth 0.62x (old arms, av ~ 4.25 from A_W149) -> 0.983x
R_source pull -13.10 -> -5.25
The new text says explicitly NOT to read the surviving -5 sigma as
convention-explained, because that is not established. It also drops the
per-event theta_star-bias column, which was the wrong quantity to tabulate.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ab194/av_true— the first run with the colour-anchoredavprior (9.01 ± 0.3,ptde, Rhat 1.00, ESS 23–25k, 78/78 chains in the good region) — contradicts the caveatdc18_truth_table.pyprints, so this rewrites it rather than leaving it in place.What it claimed, and what was measured
The caveat said
theta_staris biased low by10**(-0.2*grey)= 0.766x on event 194, read straight off the challenge's own colour–surface-brightness relation. Measured: 0.983x.The arithmetic is right about the CSB relation and wrong about the target.
theta_staris protected, because the light curve pinsrho * theta_Eindependently of the SED. Cross-checked:rho = theta_star / theta_Ereproduces both the truth and the fittedrhofrom the scored table.Where it actually lands
R_sourceandD_sourceare each ~2.45x low but slide in lockstep, so their ratio — which istheta_star— survives while each is individually −5 sigma. The flux the grey term over-predicts is absorbed by a source made too cool (teffsed3040 K here), and the distance then follows the resulting luminosity inward.The prior fix is a real win the old text buried
What the new text is careful about
theta_star-bias column, which was the wrong quantity to tabulate.The matching correction is in the notes repo (
paper_extinction_convention.txt, section 4b), and issue #303 carries a comment so the paper agent does not write the retracted version.Docs/script only — no library code, no test fixtures.