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

Add UC Davis camera support to obs_lsst

    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:

                Summary Panel