Repository navigation
docs: dev's pixel columns fall back to the image itself when their background fails (socsims-1021, rapid PR 500) - #143
Merged
Conversation
…ckground fails; run 158 lost 6 of 1,998 to the failure (socsims-1021, rapid PR 500) 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
socsims-1021 task 3, the companion to rapid PR 500. The ruling
decision-pixel-columns-failsaid a failed 64 px local background fails the difference job, wheredevmeasures the pixel columns on the unsubtracted image. On run 158 (Russ's 10/21 socsims window,dev's socsims SFFT block) that failed 6 of 1,998 differences, all SCA 18; Russ's run of the same images exited 0 on all 1,998. Rapid PR 500 takesdev's fallback. This rewrites the ruling to match (labeldecision-pixel-columns-background), along with the difference page's sentence and the two migration-table references.Files
system/decisions.md: the ruling rewritten in place, its label renamed, the run 158 measurement added.system/difference.md: the sentence and its reference.system/migration.md: two references to the renamed label.Divergences
The ruling is no longer a divergence from
dev. It stays on the page because it carries the background's measurements and the history behind rapid PR 500.Measurements
Run 158: 6 of 1,998 window differences failed with photutils'
All boxes contain <= 3686.4 unmasked or finite pixels. A central 64 px box of job 28596's SFFT difference keeps 0.885 of its pixels after the 3-sigma clip, under photutils' 0.90. The raw numbers are in rapid PR 500's body.Improvements seen
none
Session notes
none
What the next task must know
Task 5's page cites
decision-pixel-columns-background.Anything left
none
🤖 Generated with Claude Code