Repository navigation
decisions, difference, psf: the reference PSF measured in the difference job (fixes refpsf) - #94
Merged
Merged
Conversation
…d in the difference job Co-Authored-By: Claude Opus 5.5 <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.
What and why
fixes-2026-10-23 task 3 (refpsf), the docs half of Caltech-IPAC/rapid#344.
system/decisions.md:decision-reference-psf-measured, one ruling: the reference PSF is derived data measured in the difference job, recorded in its manifest, not an input chosen at launch; the reason is run 17's 15% width difference, the cause of 73% of the null test's detections; no fallback; a divergence fromdev's static per-detector file.system/difference.md: three inputs (the science PSF only); the measurement, its cuts, the exit-65 refusal and themetricskeys;refimage_psf.fits.gzamong the kept files.system/psf.md: one file and one row, the science kind; the "reference PSF rule" open question closed by pointing at the difference page.Divergences
None from the brief. The cut set on the difference page includes the width cut the code PR adds (the fixes ledger's sharpness and roundness values were not recorded; see #344).
Measurements
The difference page quotes the local measurement on run 17's pass 4 reference (1.080 px from 38 stars, 4.6 s, science 0.947); the Batch proof is in the runbook PR.
Improvements seen
system/psf.md's "Not decided here" still discusses per-quadrant PSFs for the science image; the same question now applies to the measured reference PSF (sigma varies by 0.03 px across the frame), and one paragraph on the difference page could own both.Session notes
See the runbook PR.
🤖 Generated with Claude Code