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

Include obs_decam into official LSST repositories

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: obs_decam
    • Labels:
      None
    • Team:
      SQuaRE

      Description

      As DECam data will be one of the precursor data sets used to assess the quality of LSST code, we need this module in LSST-controlled repositories.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment - - edited

            Can we split this ticket into multiple items (or else limit its scope if covered by existing tickets). I see these distinct items here:

            1. Add obs_decam to LSST github
            2. Add obs_decam to Jenkins
            3. Add ISR to obs_decam
            4. Make it easy to use.

            #1 is done and #3 is being worked on. #4 is a bit vague.

            Show
            tjenness Tim Jenness added a comment - - edited Can we split this ticket into multiple items (or else limit its scope if covered by existing tickets). I see these distinct items here: 1. Add obs_decam to LSST github 2. Add obs_decam to Jenkins 3. Add ISR to obs_decam 4. Make it easy to use. #1 is done and #3 is being worked on. #4 is a bit vague.
            Hide
            hchiang2 Hsin-Fang Chiang added a comment -

            Sounds to me that #3 is independent and not covered by this ticket, and can be removed from here. But I'm not completely sure what Mario Juric meant in the first place.

            Show
            hchiang2 Hsin-Fang Chiang added a comment - Sounds to me that #3 is independent and not covered by this ticket, and can be removed from here. But I'm not completely sure what Mario Juric meant in the first place.
            Hide
            nidever David Nidever [X] (Inactive) added a comment -

            Isn't DECam ISR covered by DM-4191?

            Show
            nidever David Nidever [X] (Inactive) added a comment - Isn't DECam ISR covered by DM-4191 ?
            Hide
            tjenness Tim Jenness added a comment - - edited

            Yes, I added a related link. I am asking that the scope of this ticket be clarified so it is clear when the work in the ticket is completed. If this is actually meant to be an epic containing other work tickets rather than a story then that is also fine.

            It seems that #2 and #3 are already covered by tickets and #1 is done.

            Show
            tjenness Tim Jenness added a comment - - edited Yes, I added a related link. I am asking that the scope of this ticket be clarified so it is clear when the work in the ticket is completed. If this is actually meant to be an epic containing other work tickets rather than a story then that is also fine. It seems that #2 and #3 are already covered by tickets and #1 is done.
            Hide
            hchiang2 Hsin-Fang Chiang added a comment -

            Yes, currently we can do ISR to the extent as described in DM-4191. But I don't think we need a fully functional ISR for obs_decam to be included in Jenkins. The descriptions sound to me that it's happy enough to take data preprocessed by the Community Pipeline.

            (p.s. I'm not defining what is "fully functional" either...)

            Show
            hchiang2 Hsin-Fang Chiang added a comment - Yes, currently we can do ISR to the extent as described in DM-4191 . But I don't think we need a fully functional ISR for obs_decam to be included in Jenkins. The descriptions sound to me that it's happy enough to take data preprocessed by the Community Pipeline. (p.s. I'm not defining what is "fully functional" either...)
            Hide
            tjenness Tim Jenness added a comment -

            Now that I re-read the original ticket description I see that it was saying that ISR would not be included to start with. Maybe we just close this ticket on the basis that the repo is now official (and tweak the description accordingly).

            Show
            tjenness Tim Jenness added a comment - Now that I re-read the original ticket description I see that it was saying that ISR would not be included to start with. Maybe we just close this ticket on the basis that the repo is now official (and tweak the description accordingly).
            Hide
            nidever David Nidever [X] (Inactive) added a comment -

            sounds good to me

            Show
            nidever David Nidever [X] (Inactive) added a comment - sounds good to me
            Hide
            tjenness Tim Jenness added a comment -

            The repository is now part of the LSST github org.

            Show
            tjenness Tim Jenness added a comment - The repository is now part of the LSST github org.

              People

              Assignee:
              nidever David Nidever [X] (Inactive)
              Reporter:
              mjuric Mario Juric
              Watchers:
              David Nidever [X] (Inactive), Hsin-Fang Chiang, Mario Juric, Simon Krughoff, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.