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

Change imSim/PhoSim gen3 instrument class names to be more explicit

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Team:
      Architecture

      Description

      In DM-26811 we changed the instrument names from (e.g.) LSST-ImSim to LSSTCam-imSim. This made the instrument more specific and fixed a case problem. By specifying LSSTCam specifically, it will be distinguished from (e.g). ComCam and hopefully will also shadow the real instruments.

      I suggest we also change the class names to match. So for example

      LsstImSim(LsstCam):

      would change to

      LsstCamImSim(LsstCam):

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Chris Walter please take a look. I change the translator classes (which is an internal detail) and the instrument classes.

            Show
            tjenness Tim Jenness added a comment - Chris Walter please take a look. I change the translator classes (which is an internal detail) and the instrument classes.
            Hide
            cwalter Chris Walter added a comment -

            This looks good to me.

            Show
            cwalter Chris Walter added a comment - This looks good to me.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              cwalter Chris Walter
              Reviewers:
              Chris Walter
              Watchers:
              Chris Walter, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  CI Builds

                  No builds found.