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

Add check for flag in schema before accessing

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_analysis
    • Labels:
      None
    • Story Points:
      0.25
    • Epic Link:
    • Sprint:
      DRP F18-2
    • Team:
      Data Release Production

      Description

      A bug affecting visitAnalysis.py was introduced on DM-14874 where culling of duplicate and sky objects was made.  The latter are only added in at the coadd level, so a check for the existence of this field should be made prior to accessing it to avoid the following failure:

      KeyError: "Field with name 'merge_peak_sky' not found"

        Attachments

          Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. DM-14874 Add options to select against duplicate coadd sources

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            lauren Lauren MacArthur added a comment -

            Would you mind giving this bug fix a quick look?  I can confirm all 5 scripts work with this fix.

            Show
            lauren Lauren MacArthur added a comment - Would you mind giving this bug fix a quick look?  I can confirm all 5 scripts work with this fix.
            Hide
            Permalink
            tmorton Tim Morton [X] (Inactive) added a comment -

            Yup, looks like a good fix to me! (I'm assuming it's just the one one-liner commit.)

            Show
            tmorton Tim Morton [X] (Inactive) added a comment - Yup, looks like a good fix to me! (I'm assuming it's just the one one-liner commit.)
            Hide
            Permalink
            lauren Lauren MacArthur added a comment -

            Indeed...sorry I forgot to make the PR...it’s there now. Will merge tomorrow. Thanks for the quick review!

            Show
            lauren Lauren MacArthur added a comment - Indeed...sorry I forgot to make the PR...it’s there now. Will merge tomorrow. Thanks for the quick review!

              People

              Assignee:
              lauren Lauren MacArthur
              Reporter:
              lauren Lauren MacArthur
              Reviewers:
              Tim Morton [X] (Inactive)
              Watchers:
              Lauren MacArthur, Tim Morton [X] (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.