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

Add obs_lsst to pipelines.lsst.io sphinx build

    Details

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

      Description

      Once RFC-569 is adopted and obs_lsst is added to lsst_distrib, the sphinx build in pipelines.lsst.io has to be modified to include obs_lsst.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            John Swinbank you removed the science pipelines label but you didn't declare which team is going to do the work on this ticket? Who usually updates pipelines.lsst.io dependencies? SQuaRE or the product owner of the package being added?

            Show
            tjenness Tim Jenness added a comment - John Swinbank you removed the science pipelines label but you didn't declare which team is going to do the work on this ticket? Who usually updates pipelines.lsst.io dependencies? SQuaRE or the product owner of the package being added?
            Hide
            swinbank John Swinbank added a comment -

            Likely I'll do it myself, but adding the “SciencePipelines” label just means it won't show up in my standard queries so I'll probably forget!

            Show
            swinbank John Swinbank added a comment - Likely I'll do it myself, but adding the “SciencePipelines” label just means it won't show up in my standard queries so I'll probably forget!
            Hide
            swinbank John Swinbank added a comment -

            (Well, I would do it myself, if I could persuade pipelines_lsst_io to build at all... https://lsstc.slack.com/archives/C2B6DQBAL/p1549329085154100)

            Show
            swinbank John Swinbank added a comment - (Well, I would do it myself, if I could persuade pipelines_lsst_io to build at all... https://lsstc.slack.com/archives/C2B6DQBAL/p1549329085154100 )
            Hide
            jsick Jonathan Sick added a comment -

            Anyone can do it really (myself as doc engineer or any other developer). The docs for the process are at https://developer.lsst.io/stack/add-a-package-to-pipelines-lsst-io.html

            Show
            jsick Jonathan Sick added a comment - Anyone can do it really (myself as doc engineer or any other developer). The docs for the process are at  https://developer.lsst.io/stack/add-a-package-to-pipelines-lsst-io.html
            Hide
            swinbank John Swinbank added a comment -

            The change on https://github.com/lsst/pipelines_lsst_io/pull/111 is trivial, and I've checked that it builds for me locally, but the Dev Guide says I should use the regular review process... so Jonathan Sick, it's all yours!

            Show
            swinbank John Swinbank added a comment - The change on https://github.com/lsst/pipelines_lsst_io/pull/111 is trivial, and I've checked that it builds for me locally, but the Dev Guide says I should use the regular review process... so Jonathan Sick , it's all yours!
            Hide
            jsick Jonathan Sick added a comment -

            Looks good, thanks.

            Show
            jsick Jonathan Sick added a comment - Looks good, thanks.
            Hide
            swinbank John Swinbank added a comment -

            Thank you! Merged & done.

            Show
            swinbank John Swinbank added a comment - Thank you! Merged & done.

              People

              • Assignee:
                swinbank John Swinbank
                Reporter:
                tjenness Tim Jenness
                Reviewers:
                Jonathan Sick
                Watchers:
                John Swinbank, Jonathan Sick, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: