Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: meas_extensions_photometryKron
-
Labels:None
-
Story Points:1
-
Epic Link:
-
Sprint:Science Pipelines DM-S15-1
-
Team:Data Release Production
Description
These are to note leftovers from DM-982. They could be done in a single issue.
1. I commented code out referring to correctfluxes, but it will need to be restored once it is available in the new framework.
2. Jim asked me to replace the computeSincFlux which is currently in PsfImage.cc in meas_algorithms with a similar call in meas_base/ApertureFlux.cc. I did not do this because it became rather complicated, and can just as easily be done when the meas_algorithms routine is moved or removed. Basically, the templating in ApertureFlux is on Pixel type, whereas in meas_algorithms it is on ImageT (where ImageT is not necessarily a single class hierarchy – e.g., Image and MaskedImage). So I left this for now.
Attachments
Issue Links
Activity
Hi Perry – my comment above is still outstanding. If changes will need to be made to _photometryKron when DM-85 is done, we need a way of tracking that. It could be this issue, or it could be another issue, but it should be clear which. If those changes don't need to be made, then we can close this issue, but we need a clear statement to that effect.
i agree. So I will just take it out of the review loop and defer it.
Perry Gee
added a comment - i agree. So I will just take it out of the review loop and defer it. Actually, I guess there is no way for me to mark it as deferred, so will just put it is a working state again and you can move it to another sprint.
Perry Gee
added a comment - Actually, I guess there is no way for me to mark it as deferred, so will just put it is a working state again and you can move it to another sprint. Since this ticket covers multiple issues, I think it makes sense to close this one down now and file a specific ticket for the aperture corrections to _photometryKron. So I've done that; new ticket is DM-2429.
John, can you move this issue to review complete or defer it, depending on whether you think that there is still work to be done on it? I think the only open issue has to do with aperture corrections.