Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-1592

Protect meas_astrom from crashing if flags in badFlags listField do not exist

    Details

      Description

      meas_astrom is crashing if flags in the badFlags listField parameters do not exist in the schema of the sources passed to the astrometry solver.

      Should check every entry in the badFlags listField and ignore it if it is not present in the source table schema. Should also print a warning.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment -

            Silently ignoring nonexistent fields sounds dangerous. I think it's better to have users update their config.

            Also, at some point catalogs in older formats will be updated to modern format as they are read in, which will make this less needed.

            I suggest closing as "wont' fix"

            Show
            rowen Russell Owen added a comment - Silently ignoring nonexistent fields sounds dangerous. I think it's better to have users update their config. Also, at some point catalogs in older formats will be updated to modern format as they are read in, which will make this less needed. I suggest closing as "wont' fix"
            Hide
            swinbank John Swinbank added a comment -

            Won't Fix, as suggested (and given the lack of activity).

            Show
            swinbank John Swinbank added a comment - Won't Fix, as suggested (and given the lack of activity).

              People

              • Assignee:
                boutigny Dominique Boutigny
                Reporter:
                boutigny Dominique Boutigny
                Watchers:
                Dominique Boutigny, John Swinbank, Russell Owen
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel