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

Include obs_decam into official LSST repositories

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • obs_decam
    • None
    • 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

            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.

            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.

            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 mjuric meant in the first place.

            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 mjuric meant in the first place.

            Isn't DECam ISR covered by DM-4191?

            nidever David Nidever [X] (Inactive) added a comment - Isn't DECam ISR covered by DM-4191 ?
            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.

            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.

            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...)

            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...)
            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).

            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).

            sounds good to me

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

            The repository is now part of the LSST github org.

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

            People

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

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.