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

Remove globals from obs_lsst

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Won't Fix
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: obs_lsst
    • Labels:
      None
    • Team:
      Data Release Production

      Description

      LsstCamMapper has some global variables for caching the camera and mapper. This is functional (for performance) but not good practice (the comments in the code agree). This may not be fixable for _camera which should be clarifed in this ticket. 

        Attachments

          Issue Links

          blocks

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-16397 Do release 17.0 of science pipelines

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-17387 Add obs_lsst to the lsst_obs table file

          • Done
          relates to

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-569 Add obs_lsst to lsst_distrib

          • Implemented
          split to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-17376 remove _camera global from obs_lsst

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            rhl Robert Lupton added a comment -

            There are two parts to this: _lsstCamMapper and _camera. The former is simple, and will be resolved first. The latter requires work on YamlCamera in obs_base

            Show
            rhl Robert Lupton added a comment - There are two parts to this: _lsstCamMapper and _camera . The former is simple, and will be resolved first. The latter requires work on YamlCamera in obs_base
            Hide
            Permalink
            rhl Robert Lupton added a comment -

            It proved simpler to just fix this along with DM-17376

            Show
            rhl Robert Lupton added a comment - It proved simpler to just fix this along with DM-17376

              People

              Assignee:
              rhl Robert Lupton
              Reporter:
              womullan Wil O'Mullane
              Reviewers:
              Wil O'Mullane
              Watchers:
              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.