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

Fix dataset lookup in multibandDriver.py

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_drivers
    • Labels:
      None
    • 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".

        Attachments

          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:

                  Jenkins Builds

                  No builds found.