Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: meas_algorithms, meas_base, pipe_tasks
-
Labels:
-
Story Points:4
-
Epic Link:
-
Sprint:Science Pipelines DM-W15-5, Science Pipelines DM-S15-1
-
Team:Data Release Production
Attachments
Issue Links
- blocks
-
DM-1766 Remove in-memory support of old-version afw::table objects
- Done
-
DM-586 Cleanup Source.h.m4
- Won't Fix
-
DM-425 Remove support for compound field types
- Invalid
-
DM-1011 Remove use of compound fields in minimal schema
- Invalid
- is blocked by
-
DM-1073 remove old forced photometry tasks
- Done
(2 relates to)
I swear that I made those debugging and cosmetic changes to measAlgs.py, PsfAttributes.cc, and ImagePsf.cc at least once before. The display=True for sure. Sorry I left that junk in.
I am not able to remove the include lsst/meas/algorithms.h from hsmLib.i without some work, so I am going to skip that, and go ahead and run buildBot on the rest. It is probably something simple, but it is not obvious to me on cursory inspection. If you can spot it, send me email. I will probably wait until later the merge, when the buildBot is done.
meas_extensions_photometryKron seems to be happy with the idea.
The comment you made in ImagePsf about the separate declaration of the declaration of the const & from the _psfImage: I was getting a type error on the call to fitCentroid which had me convinced that I had to create a const & (or create in the call, which was pretty verbose looking).
I am surprised to find that your suggestion compiles.