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-70 DRP 2-D Pipeline <p>In obs_pfs/python/lsst/obs/pfs/ingest.py replace print statements with debug info</p> PIPE2D-70

In ingest.py replace print statements with debug info

Story Major Done Done aritter aritter Thu, 18 Aug 2016 05:32:30 +0000 Fri, 24 Mar 2017 21:01:24 +0000 Fri, 24 Mar 2017 21:01:24 +0000 0 3 <p>see tickets/<a href="https://pfspipe.ipmu.jp/jira/browse/PIPE2D-70" title="In ingest.py replace print statements with debug info" class="issue-link" data-issue-key="PIPE2D-70"><del>PIPE2D-70</del></a> in obs_pfs</p> <p>Hey <a href="https://pfspipe.ipmu.jp/jira/secure/ViewProfile.jspa?name=cloomis" class="user-hover" rel="cloomis">cloomis</a>: this came up at today's meeting. Do you have an ETA on a review?</p> <p>Good news: it still rebases cleanly from master!</p> <p>But two things:</p> <ul class="alternate" type="square"> <li>Shouldn't <tt>self.log.logdebug()</tt> be <tt>self.log.debug()</tt>? Or did <tt>lsst.log.Log</tt> add logdebug at some point?</li> <li>There are (only) a few more print statements in obs_pfs: in <tt>bin.src{genCalibRegistry.py</tt>, <tt>bin.src/genDefectFits.py</tt>, and {tests/PfsFiberTrace.py}}. Should they be fixed in this ticket?</li> </ul> <p>Comments addressed. Review done.</p> <p>Addressed both points and pushed to git, pull request initiated</p> <p>Merged into master</p> Development Rank 0|ii00v2: Reviewers cloomis Sprint 2014-16 Story Points 0.1