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

bin/genCameraRepository.py broken in obs_cfht & obs_sdss

    Details

    • Type: Bug
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: obs_cfht, obs_sdss
    • Labels:
      None
    • Templates:
    • Team:
      Alert Production

      Description

      bin/genCameraRepository.py is broken on master. It fails with:

      python bin/genCameraRepository.py megacam/Full_Megacam_geom.paf megacam/camera
      WARNING: this code generates incorrect vertical overscan; see DM-5524
      Traceback (most recent call last):
        File "/Users/rowen/UW/LSST/lsstsw3/stack/DarwinX86/pex_config/13.0-3-g520d906/python/lsst/pex/config/config.py", line 335, in __set__
          self._validateValue(value)
        File "/Users/rowen/UW/LSST/lsstsw3/stack/DarwinX86/pex_config/13.0-3-g520d906/python/lsst/pex/config/config.py", line 248, in _validateValue
          raise TypeError(msg)
      TypeError: Value DetectorType.SCIENCE is of incorrect type lsst.afw.cameraGeom._detector.DetectorType. Expected type int
      

      Also note that the generated file megacam/camera/camera.py is (or was) documented to be auto-generated by that script, so don't touch. However, that file is committed and clearly has been edited, so I plan to update that text as part of DM-5922 and include a TO DO that points to this ticket.

      This ticket is a request to do the following:

      • Either fix the script or eliminate it.
      • Check all other scripts in the bin directory
      • Move any scripts that are wanted to new directory bin.src
      • Run any relevant scripts as part of scons. Remove any files that will be thus generated from git.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                rowen Russell Owen
                Watchers:
                Russell Owen, Simon Krughoff, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:

                  Summary Panel