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

            No builds found.
            mjuric Mario Juric created issue -
            mjuric Mario Juric made changes -
            Field Original Value New Value
            Rank Ranked higher
            mjuric Mario Juric made changes -
            Assignee Mario Juric [ mjuric ]
            mjuric Mario Juric made changes -
            Assignee Mario Juric [ mjuric ]
            tjenness Tim Jenness made changes -
            Component/s obs_decam [ 12851 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by RFC-74 [ RFC-74 ]
            tjenness Tim Jenness made changes -
            Assignee David Nidever [ nidever ]
            tjenness Tim Jenness made changes -
            Team SQuaRE [ 10302 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-3196 [ DM-3196 ]
            nidever David Nidever [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 13076 ]
            nidever David Nidever [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 13076 ] This issue links to "Page (Confluence)" [ 13076 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-4370 [ DM-4370 ]
            tjenness Tim Jenness made changes -
            Watchers Hsin-Fang Chiang, Mario Juric [ Hsin-Fang Chiang, Mario Juric ] Hsin-Fang Chiang, Mario Juric, Simon Krughoff [ Hsin-Fang Chiang, Mario Juric, Simon Krughoff ]
            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.
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-3790 [ DM-3790 ]
            tjenness Tim Jenness made changes -
            Link This issue relates to DM-4191 [ DM-4191 ]
            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
            tjenness Tim Jenness made changes -
            Summary Include obs_decam into official LSST repositories, and make it user friendly Include obs_decam into official LSST repositories
            tjenness Tim Jenness made changes -
            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, and we need to make sure it's easy to use on DECam data with ISR removed by the Community Pipeline. 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.
            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.
            tjenness Tim Jenness made changes -
            Resolution Done [ 10000 ]
            Status To Do [ 10001 ] Done [ 10002 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-3790 [ DM-3790 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-4370 [ DM-4370 ]
            tjenness Tim Jenness made changes -
            Link This issue blocks DM-3790 [ DM-3790 ]
            tjenness Tim Jenness made changes -
            Link This issue blocks DM-4370 [ DM-4370 ]

              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.