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

ci_hsc failure: aperture correction fields for PsfFlux not present

    Details

      Description

      ci_hsc is failing with the following error, almost certainly due to DM-15244:

      File "/j/ws/stack-os-matrix/centos-6.devtoolset-6.py3/lsstsw/build/ci_hsc/python/lsst/ci/hsc/validate.py", line 205, in validateSources
        self.checkApertureCorrections(catalog)
      File "/j/ws/stack-os-matrix/centos-6.devtoolset-6.py3/lsstsw/build/ci_hsc/python/lsst/ci/hsc/validate.py", line 117, in checkApertureCorrections
        (("%s_flag_apCorr" % alg) in catalog.schema))
      File "/j/ws/stack-os-matrix/centos-6.devtoolset-6.py3/lsstsw/build/ci_hsc/python/lsst/ci/hsc/validate.py", line 90, in assertTrue
        raise AssertionError("Failed test: %s" % description)
      AssertionError: Failed test: Aperture correction fields for base_PsfFlux are present.
      

        Attachments

          Issue Links

            Activity

            Hide
            ktl Kian-Tat Lim added a comment -

            I pushed a fix for this as f9130ea in meas_base.  But this will break compatibility of any tables generated by the incorrect code, which will include anyone using master for the last 12 hours or using the daily.  Do we need to worry about that?  I'm guessing we'd need to use up another table version to fix that problem.

            Show
            ktl Kian-Tat Lim added a comment - I pushed a fix for this as f9130ea in meas_base.  But this will break compatibility of any tables generated by the incorrect code, which will include anyone using master for the last 12 hours or using the daily.  Do we need to worry about that?  I'm guessing we'd need to use up another table version to fix that problem.
            Hide
            jbosch Jim Bosch added a comment -

            I'm not worried about that, but perhaps we should post to dm-notifications on community warning about the broken daily.  Kian-Tat Lim, did you verify that this fixed ci_hsc?  If so, I'm okay with closing this.

            Show
            jbosch Jim Bosch added a comment - I'm not worried about that, but perhaps we should post to dm-notifications on community warning about the broken daily.  Kian-Tat Lim , did you verify that this fixed ci_hsc?  If so, I'm okay with closing this.
            Hide
            rowen Russell Owen added a comment -

            Kian-Tat Lim thank you for fixing this. Your solution looks correct. My apologies for the error.

            Show
            rowen Russell Owen added a comment - Kian-Tat Lim thank you for fixing this. Your solution looks correct. My apologies for the error.
            Hide
            ktl Kian-Tat Lim added a comment - - edited

            Started 28353.  I think it will take about 6 hours.

            Show
            ktl Kian-Tat Lim added a comment - - edited Started 28353 .  I think it will take about 6 hours.
            Hide
            ktl Kian-Tat Lim added a comment -

            Jenkins passed.

            Show
            ktl Kian-Tat Lim added a comment - Jenkins passed.

              People

              • Assignee:
                rowen Russell Owen
                Reporter:
                rowen Russell Owen
                Watchers:
                Jim Bosch, Kian-Tat Lim, Lauren MacArthur, Russell Owen
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel