fix: correct U-channel imsize in sequencerd.cfg.in (match camerad, 1024x1400)#458
Open
cfremling wants to merge 1 commit into
Open
fix: correct U-channel imsize in sequencerd.cfg.in (match camerad, 1024x1400)#458cfremling wants to merge 1 commit into
cfremling wants to merge 1 commit into
Conversation
sequencerd's CAMERA_PROLOGUE imsize for the U channel (2198 4202) did not match camerad's DETECTOR_GEOMETRY for U (2048 4102), so U came out 1099x1433 when the sequencerd prologue was applied but 1024x1400 (the required size) when camerad's own geometry governed. Because the size then depended on which daemon last set the U geometry (camerad start / binning change vs sequencer startup), U flipped between 1024x1400 and 1099x1433 across a night -- causing science/cal size mismatches that broke Quick Look reduction for U (only G displayed on 2026-06-24). Set sequencerd imsize U to 2048 4102 to match camerad DETECTOR_GEOMETRY (U COL ROW 2048 4102), the required 1024x1400 at 2x3. G/R/I already matched at 1220 4114. The deployed Config/sequencerd.cfg was corrected on the instrument; this brings the tracked template in line so a rebuild does not reintroduce the mismatch. Co-Authored-By: Claude Opus 4.8 (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
1 participant
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.
Problem
sequencerd.cfg.inset the U-channel readout geometry toimsize U 2198 4202, which does not match camerad'sDETECTOR_GEOMETRY=(U COL ROW 2048 4102). The two daemons therefore disagreed on the U size:U's final size depended on which daemon last set it (camerad start / binning change vs sequencer startup), so U flipped between the two sizes across a night. On 2026-06-24 the cals came out 1024×1400 but the science came out 1099×1433, so Quick Look could not reduce U against the cals — only G displayed. (G/R/I were unaffected: their sequencerd
imsizealready matches camerad at1220 4114.)Fix
Set sequencerd
imsize U→2048 4102to match camerad'sDETECTOR_GEOMETRYand produce the required 1024×1400. The deployedConfig/sequencerd.cfgwas already corrected on the instrument; this updates the tracked template (.cfg.in) so a rebuild (CONFIGURE_FILE regenerates.cfgfrom.cfg.in) does not reintroduce the mismatch.Notes
🤖 Generated with Claude Code