Jim Bosch added a comment - 06/Nov/14 12:20 PM Moved this to the current sprint, and boosted the story point estimate for two reasons: new person working on it who is less familiar with the codebase may also need to make some non-cameraGeom modifications to obs_subaru to get it working again with the LSST pipeline

Lauren MacArthur added a comment - 22/Dec/14 5:23 PM Continuing from where Simon Krughoff left off, I have updated obs_subaru such that processCcd.py will run through past the isr stage (several ccds were tested and all made it at least to the calibrate.astrometry stage). I have pulled over as many commits from the HSC side as seemed reasonable at this stage. In particular, work needs to be done on the measurement algorithms (currently disabled for the most part). I also have not yet pulled over any of RHL's commits related to bin/plotFocus.py and bin/showDither.py. I will look into these as part of a future ticket. There are many (~85) commits since tickets/ DM-245 branched from master included. I have pushed these to branch u/lauren/ DM-245 , the last one in the log is: Last of Commits de069f3 2014-11-06 (HEAD, u/lauren/ DM-245 ) config: set priorityList for det and meas merges [Lauren MacArthur] The first few are from Simon Krughoff (not yet merged to master). The first commit by me was 8ceffc2, as shown below: First of Commits 8ceffc2 2014-11-19 config: remove all root.measurement.algorithms references outside try statements in processCcd.py and hsc/processCcd.py [Lauren MacArthur] aee9f22 2014-11-05 Removes white spaces from hscMapper.py [Simon Krughoff] bbe793b 2014-11-05 Adds comment about SUBARU_DATA [Simon Krughoff] 4db5894 2014-11-05 Adds a new utility to display cameras [Simon Krughoff] 2b62835 2014-11-04 Adds camera repository files [Simon Krughoff] d155bbf 2014-11-04 Adds a utility for building camera registries [Simon Krughoff] d0ca401 2014-11-04 Wraps distortion code in XYTransform [Simon Krughoff] cb4a8e2 2014-11-05 Updates genDefectFits.py to use new CameraGeom [Simon Krughoff] fc0a804 2014-11-05 Update Exceptions in Crosstalk to new names. [Simon Krughoff] The only other package that was modified and whose setup is required for this obs_subaru to run is ip_isr. This is from branch u/price/hsc-merge-20141120.

John Swinbank added a comment - 06/Jan/15 10:18 AM Folks, I would like to switch the "epic link" for this story to DM-1382 ("work on ideas for the deblender"). We originally defined Lauren's work under that epic as DM-1474 before realising that duplicated this story. This is basically just a book-keeping change, with two benefits: The original parent epic of this story, DM-753 , was scheduled for completion in Summer 2014. This is the only outstanding story blocking that epic from being closed. The work currently being invested in this story can be counted against value earned in W15. If anybody find this change objectionable, please let me know.

Paul Price added a comment - 19/Jan/15 4:40 PM Review was done on github: https://github.com/LSST/obs_subaru/pull/2 Lauren and I are going to meet soon to discuss how to clean some things up.

Paul Price added a comment - 20/Feb/15 11:38 AM Here's the submission from ip_isr, on tickets/ DM-245 at github. price@price-laptop:~/LSST/ip/isr (tickets/DM-245=) $ git --no-pager log --stat --reverse origin/master.. commit 33c191f382ffacc82dba5250b4f5f7f78a1af28d Author: Paul Price <price@astro.princeton.edu> Date: Wed Jul 10 09:32:29 2013 -0400   fringe: better error message for fringe unpersistence Lack of a fringe frame is often characterised by an obscure error from the depths of the butler that doens't mention anything about a fringe frame. This provides a useful error message.   python/lsst/ip/isr/fringe.py | 5 ++++- tests/testFringes.py | 3 ++- 2 files changed, 6 insertions(+), 2 deletions(-)   commit 55cf411aa20e1fd45bb857e53c50592f63e1f561 Author: Paul Price <price@astro.princeton.edu> Date: Wed Dec 4 15:50:33 2013 -0500   overscan: allow masking of overscan On HSC, saturated pixels can leak into the overscan. We mask them, but they need to be ignored. Added a statControl keyword argument for control of the rejection and masking.   python/lsst/ip/isr/isr.py | 21 +++++++++++++++------ 1 file changed, 15 insertions(+), 6 deletions(-)   commit 72841624d121205faa3e8391079d2b07a2272e05 Author: Jim Bosch <jbosch@astro.princeton.edu> Date: Thu Mar 13 10:18:35 2014 -1000   Pass NumPy arrays to afwMath.makeInterpolate.   python/lsst/ip/isr/isr.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)   commit 460b03e4c40491df5efe5aa3326c980a6fbc1ec4 Author: Paul Price <price@astro.princeton.edu> Date: Tue Apr 1 17:50:21 2014 +0000   transposeMaskedImage: fix transposition Wasn't working due to numpy float vs float32 issues with ImageF.   python/lsst/ip/isr/isr.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-)   commit c250aaceccdb95a5edabefbb2c84d8ba61596fdf Author: Robert Lupton the Good <rhl@astro.princeton.edu> Date: Tue Apr 8 16:11:48 2014 -0400   overscan: Toughen code to handle the case that no rows are masked See https://jira.lsstcorp.org/browse/DM-415. In this case all the rows were bad, but the masking code passed them all so this bug triggered   python/lsst/ip/isr/isr.py | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-)   commit cb2504fe4f4846f34aae8f90dfb617d7398bedff Author: Jim Bosch <jbosch@astro.princeton.edu> Date: Fri May 16 18:27:17 2014 -0400   Make random number generation in fringe code deterministic   python/lsst/ip/isr/fringe.py | 13 +++++++++---- tests/testFringes.py | 13 ++++++------- 2 files changed, 15 insertions(+), 11 deletions(-)   commit 9749e104108b0a391c913d30501489ed65503750 Author: Paul Price <price@astro.princeton.edu> Date: Tue Oct 7 14:55:11 2014 -0400   saturationDetection: apply to overscan as well as image The overscan may be saturated from charge bleeding in from the image.   python/lsst/ip/isr/isrTask.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)   commit 7b465785a8142b8693ca0eae5e4ab5f5392d8df1 Author: Paul Price <price@astro.princeton.edu> Date: Mon Oct 6 19:39:29 2014 -0400   defects: activate useFallbackValueAtEdge for interpolation Lack of use of this feature meant that NANs were not being interpolated over.   python/lsst/ip/isr/isr.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)