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-207 DRP 2-D Pipeline <p>There's a lot of unused code in drp_stella and it needs to be removed. Let us start by running a python coverage tool on all the "live" command line tasks, and remove all code that doesn't get used (subject to some sort of sanity test). This should cover unused swig wrappers as well as unused python code.</p> <p>Then, after that, run a C++ coverage tool (e.g. tcov) and remove all C++ that isn't used.</p> PIPE2D-207

Run coverage tools and remove *all* unused code

Epic Major Done Done Unassigned rhl Wed, 21 Jun 2017 16:02:37 +0000 Wed, 26 Jul 2017 18:37:08 +0000 Wed, 26 Jul 2017 18:37:08 +0000 0 1 <p>Started on this. Removed unused python and merged.</p> <p>If we need to remove more code, please call the branch <tt>tickets/<a href="https://pfspipe.ipmu.jp/jira/browse/PIPE2D-207" title="Run coverage tools and remove *all* unused code" class="issue-link" data-issue-key="PIPE2D-207"><del>PIPE2D-207</del></a>a</tt></p> <p>Removed LinFitBevingtonEigen on branch <tt>tickets/<a href="https://pfspipe.ipmu.jp/jira/browse/PIPE2D-207" title="Run coverage tools and remove *all* unused code" class="issue-link" data-issue-key="PIPE2D-207"><del>PIPE2D-207</del></a>a</tt>; please use <tt>tickets/<a href="https://pfspipe.ipmu.jp/jira/browse/PIPE2D-207" title="Run coverage tools and remove *all* unused code" class="issue-link" data-issue-key="PIPE2D-207"><del>PIPE2D-207</del></a>b</tt> next.</p> <p>Removed <tt>makeNormFlatFiberTrace</tt> and <tt>getRawPointer</tt> on branch tickets/<a href="https://pfspipe.ipmu.jp/jira/browse/PIPE2D-207" title="Run coverage tools and remove *all* unused code" class="issue-link" data-issue-key="PIPE2D-207"><del>PIPE2D-207</del></a>b; please use tickets/<a href="https://pfspipe.ipmu.jp/jira/browse/PIPE2D-207" title="Run coverage tools and remove *all* unused code" class="issue-link" data-issue-key="PIPE2D-207"><del>PIPE2D-207</del></a>c next.</p> <p>N.b. didn't remove <tt>fiberTraceCoordinatesRelativeTo</tt> as it's used internally, and <tt>ccdToFiberTraceCoordinates</tt> is used to test it (both are bound to python). These need to go as part of <tt>FiberTrace</tt> cleanup.</p> <p>merged</p> Duplicate PIPE2D-221 Development Epic Color ghx-label-6 Epic Name Remove unused code from drp_stella Epic Status Rank 0|ii028f: