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

remove _camera global from obs_lsst

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • obs_lsst
    • None
    • DM Science

    Description

      Splitting the _camera work from DM-17291 as it's separate (and more complex) than the _lsstCamMapper work.

      Attachments

        Issue Links

          Activity

            rhl Robert Lupton added a comment - - edited

            Note that there is also a DM-17376 branch for obs_base, and you need the DM-17446 branch of ip_isr to run ci_lsst.  I think that both PRs exists, although Jira doesn't seem to have noticed

            rhl Robert Lupton added a comment - - edited Note that there is also a DM-17376 branch for obs_base, and you need the  DM-17446 branch of ip_isr to run ci_lsst.  I think that both PRs exists, although Jira doesn't seem to have noticed
            tjenness Tim Jenness added a comment -

            I can't see a DM-17446 branch in ip_isr (or a pull request). There is a u/rhl/... branch though.

            tjenness Tim Jenness added a comment - I can't see a DM-17446 branch in ip_isr (or a pull request). There is a u/rhl/... branch though.
            tjenness Tim Jenness added a comment -

            I have reviewed the obs_base branch and obs_lsst branch. I can't find a ip_isr branch. The obs_lsst changes look okay in general but there are many rebase errors that need fixing (I had touched most of the classes you changed). I also have a problem with the VisitInfo logic.

            tjenness Tim Jenness added a comment - I have reviewed the obs_base branch and obs_lsst branch. I can't find a ip_isr branch. The obs_lsst changes look okay in general but there are many rebase errors that need fixing (I had touched most of the classes you changed). I also have a problem with the VisitInfo logic.

            I thought that czw had pushed it.  Please use u/rhl/overscanNumLeadingColumnsToSkip for testing – these fixes are independent of the work on this ticket.

            rhl Robert Lupton added a comment - I thought that czw had pushed it.  Please use u/rhl/overscanNumLeadingColumnsToSkip for testing – these fixes are independent of the work on this ticket.

            Merged to master

            rhl Robert Lupton added a comment - Merged to master

            People

              rhl Robert Lupton
              rhl Robert Lupton
              Tim Jenness
              Colin Slater, John Swinbank, Leanne Guy, Robert Lupton, Tim Jenness, Wil O'Mullane
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.