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

Add faro steps to DRP.yaml in obs_lsst/imsim

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: faro
    • Labels:

      Description

      In order to integrate `faro` metric calculation into the DP0.2 workflow, we need to update the DRP.yaml file in obs_lsst/imsim. This should include `faro` subsets that can be run as soon as the required inputs are available (e.g., metrics that only require singleFrame processing can be calculated before moving on to coadds). 

        Attachments

          Issue Links

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-32029 Add faro steps to DRP.yaml in obs_subaru

          • Done
          mentioned in

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            4 older comments
            Hide
            Permalink
            jcarlin Jeffrey Carlin added a comment -

            Both Jenkins jobs are still running, so merge is pending successful completion of those (and approval upon your review, of course).

            Show
            jcarlin Jeffrey Carlin added a comment - Both Jenkins jobs are still running, so merge is pending successful completion of those (and approval upon your review, of course).
            Hide
            Permalink
            erykoff Eli Rykoff added a comment -

            The ci_imsim needs a PR for me to review. Once that's there I can check all the boxes.

            Show
            erykoff Eli Rykoff added a comment - The ci_imsim needs a PR for me to review. Once that's there I can check all the boxes.
            Hide
            Permalink
            jcarlin Jeffrey Carlin added a comment -

            Sorry 'bout that – PR is here.

            Show
            jcarlin Jeffrey Carlin added a comment - Sorry 'bout that – PR is here .
            Hide
            Permalink
            erykoff Eli Rykoff added a comment -

            Looks good, and thanks to Dan Taranu for catching the necessary ci_imsim update. Fingers crossed for a clean jenkins!

            Show
            erykoff Eli Rykoff added a comment - Looks good, and thanks to Dan Taranu for catching the necessary ci_imsim update. Fingers crossed for a clean jenkins!
            Hide
            Permalink
            jcarlin Jeffrey Carlin added a comment -

            Both Jenkins runs are good, and both PRs merged.

            Show
            jcarlin Jeffrey Carlin added a comment - Both Jenkins runs are good, and both PRs merged.

              People

              Assignee:
              jcarlin Jeffrey Carlin
              Reporter:
              jcarlin Jeffrey Carlin
              Reviewers:
              Eli Rykoff
              Watchers:
              Dan Taranu, Eli Rykoff, Jeffrey Carlin
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.