DiaSource centroids outside of image causing crash in cuttout creation in PackageAlerts.

Description

A recent run of the ap_verify_hits2015 dataset is throwing errors with points being outside of the ccd bounding box. Likely this is coming from DiaForcedSource photometry measurements where the previous behavior was to run a NaN value instead of crashing.

Ticket is to investigate (confirm where the error is from) and fix the problem.

[Updated title. See below]

Checklist

Lucidchart Diagrams

Issue Matrix

hide

Activity

Show:
Ian Sullivan
April 5, 2021 at 11:01 PM

Looks good, please see my comments on the PR. My main question is whether we should catch all InvalidParameterErrors, or only the ones that were caused by this particular bug. I assume we would want to catch them all, but we might want to hit them for now so that we can try to understand where they are coming from.

morrisoncb
March 29, 2021 at 8:55 PM
(edited)

Initially the thought was that this problem was occurring in DiaForcedSource based on the output error. However, this error appears to be thrown by PackageAlertsTask when center of the BBox is off image. This is caused (strangely) by the DiaSource centroid itself being off image in the diffIm. I'll add some sort of try, except clause to catch this for now (likely return None for the image cuttout), however it might be worth looking into why this centroid is off image. This may be related to previous issues (https://rubinobs.atlassian.net/browse/DM-26032#icft=DM-26032) discovered in processing where negative detections some times contained peaks that were 100s or 1000s of pixels outside of the source footprint.

For now I'll test the None return and add a warning if it is used. I'll also check a few other failures to see  if it is caused by similar problems with the DiaSource centroid.

 

Done
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Reviewers

Ian Sullivan

Story Points

RubinTeam

Components

Sprint

None

Checklist

Created February 25, 2021 at 10:24 PM
Updated March 8, 2023 at 9:43 PM
Resolved April 6, 2021 at 8:16 PM