New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DM-30926: Add SkySources to ImageDifferenceTask #118
Conversation
Debug skySource code. Move around diaSource mask on sky_source flag.
d384610
to
2c894fc
Compare
@@ -85,6 +85,11 @@ class TransformDiaSourceCatalogConfig(pipeBase.PipelineTaskConfig, | |||
"data", | |||
"DiaSource.yaml") | |||
) | |||
doRemoveSkySources = pexConfig.Field( | |||
dtype=bool, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please state default explicitly. Shouldn't it be the same as doSkySources
in ImageDifferenceTask
? Especially because this is enforced in ApPipe.yaml
.
@@ -223,7 +233,7 @@ def computeBBoxSizes(self, inputCatalog): | |||
footprintBBox.minY - recY])))) | |||
if bboxSize > maxSize: | |||
bboxSize = maxSize | |||
outputBBoxSizes[idx] = bboxSize | |||
outputBBoxSizes.append(bboxSize) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
outputBBoxSizes
is now a list. Either update signature in docstring or convert to ndarray before return.
@@ -205,8 +212,11 @@ def computeBBoxSizes(self, inputCatalog): | |||
outputBBoxSizes : `numpy.ndarray`, (N,) | |||
Array of bbox sizes. | |||
""" | |||
outputBBoxSizes = np.empty(len(inputCatalog), dtype=int) | |||
outputBBoxSizes = [] | |||
for idx, record in enumerate(inputCatalog): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
idx
is not used any more. Please remove.
Make clear default in config. Update doc. Remove unused variables. Fix typo
12131a5
to
d03ed09
Compare
No description provided.