https://pfspipe.ipmu.jp/jira This file is an XML representation of an issue en-us 8.3.4 803005 13-09-2019 https://pfspipe.ipmu.jp/jira/browse/PIPE2D-1097 DRP 2-D Pipeline <p>Please include the readNoise in the variance estimates for H4RGs (n-band)</p> PIPE2D-1097

Include read noise in estimate of variance for H4RG detectors

Story Normal Done Done rhl rhl NIR Tue, 18 Oct 2022 16:19:57 +0000 Tue, 7 Feb 2023 22:36:32 +0000 Sat, 29 Oct 2022 15:50:04 +0000 0 2 <p><a href="https://github.com/Subaru-PFS/obs_pfs/pull/133" class="external-link" rel="nofollow">https://github.com/Subaru-PFS/obs_pfs/pull/133</a></p> <p>Integration test is failing with errors:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java"> File <span class="code-quote">"/scratch/pprice/jenkins/integrationTest/454/obs_pfs/python/lsst/obs/pfs/pfsMapper.py"</span>, line 361, in std_raw raise RuntimeError(f<span class="code-quote">"Rewrite me now we have {len(exp.getDetector())} channels"</span>) RuntimeError: Rewrite me now we have 8 channels </pre> </div></div> <p>See <tt>/scratch/pprice/jenkins/integrationTest/454</tt>.</p> <p>Did something not get rebased? camera.yaml only has one amplifier these days</p> <p>I think <tt>b</tt> and <tt>r</tt> arms are getting into that bit you intended for <tt>n</tt> only, and they have multiple amplifiers.</p> <p>Aargh. Yes. Thanks.</p> <p>Passed tests, merged, pushed</p> Development Rank 0|02qph7:0d Reviewers price Sprint 2DDRP-2023 A