Details

    • Type: Bug
    • Status: Done
    • Priority: Major
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_drivers
    • Labels:
      None
    • Templates:
    • Story Points:
      0.5
    • Epic Link:
    • Sprint:
      DRP F16-4, DRP F16-5
    • Team:
      Data Release Production

      Description

      MultibandDriver builds a patch reference list by checking for the existence of self.config.coaddName + "Coadd". However, when coadd assembly is performed by running coaddDriver, this coadd is not persisted in favor of only persisting the updated calexp version. Thus, currently MultibandDriver ends up with and empty patch list and exits quietly. The existence check should be on the self.config.coaddName + "Coadd_calexp".

        Issue Links

          Activity

          Hide
          erykoff Eli Rykoff added a comment -

          Code produces expected output, passes all tests.

          Show
          erykoff Eli Rykoff added a comment - Code produces expected output, passes all tests.
          Hide
          jsick Jonathan Sick added a comment - - edited

          Jonathan Sick this needs to go in the developer guide:

          Thanks Tim. I agree that ci.lsst.codes needs to be documented, especially given the UX. I'll bring it up at SQuaRE's standup to see how we can schedule it.

          Show
          jsick Jonathan Sick added a comment - - edited Jonathan Sick this needs to go in the developer guide: Thanks Tim. I agree that ci.lsst.codes needs to be documented, especially given the UX. I'll bring it up at SQuaRE's standup to see how we can schedule it.
          Hide
          swinbank John Swinbank added a comment -

          Lauren MacArthur – looks like something that should go in the release notes, right? Could you add a brief mention to https://confluence.lsstcorp.org/display/DM/Data+Release+Production+WIP+F16+Release+Notes please?

          Show
          swinbank John Swinbank added a comment - Lauren MacArthur – looks like something that should go in the release notes, right? Could you add a brief mention to https://confluence.lsstcorp.org/display/DM/Data+Release+Production+WIP+F16+Release+Notes please?
          Hide
          lauren Lauren MacArthur added a comment -

          Done.

          Show
          lauren Lauren MacArthur added a comment - Done.
          Hide
          swinbank John Swinbank added a comment -

          thank you!

          Show
          swinbank John Swinbank added a comment - thank you!

            People

            • Assignee:
              lauren Lauren MacArthur
              Reporter:
              lauren Lauren MacArthur
              Reviewers:
              Eli Rykoff
              Watchers:
              Eli Rykoff, Jim Bosch, John Swinbank, Jonathan Sick, Lauren MacArthur, Tim Jenness
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development

                  Agile