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

Generate camera description at build time

    Details

    • Story Points:
      3
    • Sprint:
      Alert Production F16 - 8, Alert Production F16 - 9, Alert Production F16 - 10
    • Team:
      Alert Production

      Description

      Camera geometry used to be defined using PAF (policy) files, which are now deprecated. As part of the transition to the refactored camera geometry scheme, scripts were introduced to convert from the PAF files to the new camera geometry configuration scheme which uses FITS files and a python file to describe the camera. These scripts are still part of the obs_* packages, and some people rely on them for making changes to the camera description. On the other hand, the generated FITS files and python file are also first-class members of the obs_* packages. This means that we have two sources of the same information, which is dangerous.

      For obs_lsstSim, obs_decam, obs_cfht and obs_sdss, we want these scripts to be the primary source of information. This means we should delete the generated files, and create them at build time. We should also standardise the name of the script used to generate these.

        Attachments

          Issue Links

            Activity

            Hide
            price Paul Price added a comment -

            Building the camera description for LSST requires a GainFile and phosimVersion. The latter appears to be just some string, but the GainFile is real content, but not present in obs_lsstSim.

            Show
            price Paul Price added a comment - Building the camera description for LSST requires a GainFile and phosimVersion . The latter appears to be just some string, but the GainFile is real content, but not present in obs_lsstSim.
            Hide
            swinbank John Swinbank added a comment -

            John Parejko, can you confirm you're working on this? If so, we'll need to move it to an appropriate epic in 02C.03.

            Show
            swinbank John Swinbank added a comment - John Parejko , can you confirm you're working on this? If so, we'll need to move it to an appropriate epic in 02C.03.
            Hide
            Parejkoj John Parejko added a comment -

            Yes, I'm just starting this (and its sibling, DM-6349).

            Show
            Parejkoj John Parejko added a comment - Yes, I'm just starting this (and its sibling, DM-6349 ).
            Hide
            swinbank John Swinbank added a comment -

            Thanks. Switching team to AP and epic to DM-5691, which looks plausible — John Parejko or Simon Krughoff, please feel free to refile as you see fit.

            Show
            swinbank John Swinbank added a comment - Thanks. Switching team to AP and epic to DM-5691 , which looks plausible — John Parejko or Simon Krughoff , please feel free to refile as you see fit.
            Hide
            krughoff Simon Krughoff added a comment -

            Good enough. Sorry for not switching it before.

            Show
            krughoff Simon Krughoff added a comment - Good enough. Sorry for not switching it before.
            Hide
            swinbank John Swinbank added a comment -

            I think this is pretty obsolete now — see also comments on DM-6349 — so I'm closing as won't fix.

            Show
            swinbank John Swinbank added a comment - I think this is pretty obsolete now — see also comments on DM-6349 — so I'm closing as won't fix.

              People

              • Assignee:
                Parejkoj John Parejko
                Reporter:
                price Paul Price
                Watchers:
                John Parejko, John Swinbank, Paul Price, Simon Krughoff
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel