Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-74

Move obs_decam under LSST umbrella

    XMLWordPrintable

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None
    • Location:
      comments

      Description

      I propose that we move obs_decam under the lsst umbrella like obs_sdss. This entails:

      • Moving it to: github.com/lsst/obs_decam
      • Adding it to buildbot's build list
      • Pipe dream: Use a decam image in an integration test

      Our current process is not efficient: one or two people keep up with every change to the stack and learn how to incorporate those changes into obs_decam (or they fall behind and spend substantial time updating it). In contrast, it is typically straightforward for the author of the original change to duplicate the update that they make on obs_sdss.

      The repository is not ready to move yet. If this proposal is adopted, we would fix up the unit test that depends on the lost testdata_decam and add some more unit tests. (The most up to date copy is under github.com/yalsayyad/obs_decam)

        Attachments

          Issue Links

            Activity

            Hide
            yusra Yusra AlSayyad added a comment -

            The tests shouldn't depend on the test repo: DM-3868. If the entry is in the ups/table, feel free to remove it.

            Show
            yusra Yusra AlSayyad added a comment - The tests shouldn't depend on the test repo: DM-3868 . If the entry is in the ups/table, feel free to remove it.
            Hide
            krughoff Simon Krughoff added a comment -

            It's listed as setupOptional. I think this is actually the right way to do things, but lsstsw doesn't ignore packages that are setupOptional and unavailable. Do you really want me to remove that line?

            Show
            krughoff Simon Krughoff added a comment - It's listed as setupOptional. I think this is actually the right way to do things, but lsstsw doesn't ignore packages that are setupOptional and unavailable. Do you really want me to remove that line?
            Hide
            hchiang2 Hsin-Fang Chiang added a comment -

            If wanted, testdata_decam is at lsst-dev:/lsst8/testdata_decam

            Show
            hchiang2 Hsin-Fang Chiang added a comment - If wanted, testdata_decam is at lsst-dev:/lsst8/testdata_decam
            Hide
            ktl Kian-Tat Lim added a comment -

            Once git-lfs is officially approved, we can store testdata_decam there.

            Show
            ktl Kian-Tat Lim added a comment - Once git-lfs is officially approved, we can store testdata_decam there.
            Hide
            swinbank John Swinbank added a comment -

            Suggest we can regard this as implemented once obs_decam is included in lsst_obs (see DM-8256, which is blocking the 13.0 release).

            Show
            swinbank John Swinbank added a comment - Suggest we can regard this as implemented once obs_decam is included in lsst_obs (see DM-8256 , which is blocking the 13.0 release).

              People

              Assignee:
              yusra Yusra AlSayyad
              Reporter:
              yusra Yusra AlSayyad
              Watchers:
              Hsin-Fang Chiang, John Swinbank, Kian-Tat Lim, Russell Owen, Simon Krughoff, Tim Jenness, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                  Jenkins

                  No builds found.