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

ap_verify HSC Gen 3 ingestion crashes on missing defineVisits config

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ap_verify, obs_subaru
    • Labels:
      None
    • Story Points:
      1
    • Epic Link:
    • Sprint:
      AP S21-6 (May)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      DM-30030 removed obs_subaru:defineVisits.py, but left a reference inside datasetIngest-gen3.py. Remove this reference and confirm that everything builds.

        Attachments

          Issue Links

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-30030 Make calibration source selection consistent between gen2 and gen3 middleware - part deux

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Hi Kian-Tat Lim, can you do a quick review of this? I've confirmed this change fixes ap_verify on lsst_devl.

            Show
            krzys Krzysztof Findeisen added a comment - Hi Kian-Tat Lim , can you do a quick review of this? I've confirmed this change fixes ap_verify on lsst_devl .
            Hide
            Permalink
            lauren Lauren MacArthur added a comment -

            I got it. Thanks for the quick weekend fix (and many apologies for the breakage...)

            Show
            lauren Lauren MacArthur added a comment - I got it. Thanks for the quick weekend fix (and many apologies for the breakage...)
            Hide
            Permalink
            ktl Kian-Tat Lim added a comment -

            Show
            ktl Kian-Tat Lim added a comment -
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Thanks for the quick weekend review!

            Show
            krzys Krzysztof Findeisen added a comment - Thanks for the quick weekend review!

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              krzys Krzysztof Findeisen
              Reviewers:
              Lauren MacArthur
              Watchers:
              Kian-Tat Lim, Krzysztof Findeisen, Lauren MacArthur
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.