Unhelpful "cannot iterate over NoneType" ImageDifferenceTask error

Description

If you do not specify the exact correct dataId (e.g., visit and ccdnum and filter, for decam), ImageDifferenceTask will fail in a very confusing way. An example traceback thanks to is pasted below.

Instead, the Task should prompt the user to specify whatever is missing from the dataId, and/or behave more like processCcd where it just figures it out.

 

Checklist

Lucidchart Diagrams

Issue Matrix

hide

Activity

Show:

Ian Sullivan 
March 29, 2021 at 6:27 PM

Marking as Invalid now that we are converting to Gen 3 and this is a Gen 2 bug.

John Swinbank 
February 15, 2019 at 10:09 AM

Butler in “confusing” shocker!

More seriously, I agree that this is not good behaviour, but I suggest it's not something we should address separately — all of the data access in ip_diffim will need to be reworked over the next few months for BG3, and which is (...should be) intrinsically less user-hostile anyway. Let's keep this ticket open so we can check that the new version really does do better.

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

Details

Assignee

Reporter

Story Points

RubinTeam

Components

Checklist

Created February 14, 2019 at 7:29 PM
Updated March 29, 2021 at 6:27 PM
Resolved March 29, 2021 at 6:27 PM