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

Add UC Davis camera support to obs_lsst

    XMLWordPrintable

    Details

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

      Description

      In obs_lsst #43 Craig Lage has added support for the UC Davis camera to obs_lsst.

      We need to shepherd this through adoption into the DM codebase.
       

        Attachments

          Activity

          Hide
          tjenness Tim Jenness added a comment -

          Merlin Fisher-Levine would you mind looking at this? We've added tests and sorted out the camera yaml file. I think I'd rather the addition of tests to ci_lsst be done on a different ticket (and preferably by someone else). Merging this to obs_lsst won't affect other people.

          Show
          tjenness Tim Jenness added a comment - Merlin Fisher-Levine would you mind looking at this? We've added tests and sorted out the camera yaml file. I think I'd rather the addition of tests to ci_lsst be done on a different ticket (and preferably by someone else). Merging this to obs_lsst won't affect other people.
          Hide
          tjenness Tim Jenness added a comment -

          Robert Lupton are you able to review this ticket? It should be relatively quick since you know the system.

          Show
          tjenness Tim Jenness added a comment - Robert Lupton are you able to review this ticket? It should be relatively quick since you know the system.
          Hide
          rhl Robert Lupton added a comment -

          I can do the review

          Show
          rhl Robert Lupton added a comment - I can do the review
          Hide
          rhl Robert Lupton added a comment -

          The code looks good. I'm still looking at adding some of the data to ci_lsst, but this need not prevent us from merging

          Show
          rhl Robert Lupton added a comment - The code looks good. I'm still looking at adding some of the data to ci_lsst, but this need not prevent us from merging
          Hide
          tjenness Tim Jenness added a comment -

          Merged. Thank you Robert Lupton for the review and thank you Craig Lage for starting this off with some code and helping out with my changes.

          Show
          tjenness Tim Jenness added a comment - Merged. Thank you Robert Lupton for the review and thank you Craig Lage for starting this off with some code and helping out with my changes.

            People

            Assignee:
            tjenness Tim Jenness
            Reporter:
            swinbank John Swinbank
            Reviewers:
            Robert Lupton
            Watchers:
            Craig Lage, John Swinbank, Merlin Fisher-Levine, Robert Lupton, Simon Krughoff, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.