Details
-
Type:
Story
-
Status: Won't Fix
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: obs_cfht, obs_decam, obs_lsstSim, obs_sdss
-
Labels:None
-
Story Points:3
-
Epic Link:
-
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
- is triggered by
-
- Adopted
Activity
Paul Price
created issue -
Paul Price
made changes -
| Epic Link |
|
| Sprint | DRP F16-1 [ 225 ] | |
| Team | Data Release Production [ 10301 ] |
| Sprint | DRP F16-1 [ 225 ] | DRP F16-2 [ 231 ] |
| Sprint | DRP F16-2 [ 231 ] | DRP F16-3 [ 237 ] |
Paul Price
made changes -
| Status | To Do [ 10001 ] | In Progress [ 3 ] |
| Assignee | Paul Price [ price ] | John Parejko [ parejkoj ] |
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.
Yes, I'm just starting this (and its sibling, DM-6349).
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.
| Team | Data Release Production [ 10301 ] | Alert Production [ 10300 ] |
Good enough. Sorry for not switching it before.
| Sprint | DRP F16-3 [ 237 ] | Alert Production F16 - 8 [ 245 ] |
| Rank | Ranked lower |
| Sprint | Alert Production F16 - 8 [ 245 ] | Alert Production F16 - 8, Alert Production F16 - 9 [ 245, 247 ] |
| Rank | Ranked higher |
| Sprint | Alert Production F16 - 8, Alert Production F16 - 9 [ 245, 247 ] | Alert Production F16 - 8, Alert Production F16 - 9, Alert Production F16 - 10 [ 245, 247, 284 ] |
| Rank | Ranked higher |
| Sprint | Alert Production F16 - 8, Alert Production F16 - 9, Alert Production F16 - 10 [ 245, 247, 284 ] | Alert Production F16 - 8, Alert Production F16 - 9, Alert Production F16 - 10, Alert Production F16 - 11 [ 245, 247, 284, 289 ] |
| Sprint | Alert Production F16 - 8, Alert Production F16 - 9, Alert Production F16 - 10, Alert Production F16 - 11 [ 245, 247, 284, 289 ] | Alert Production F16 - 8, Alert Production F16 - 9, Alert Production F16 - 10 [ 245, 247, 284 ] |
| Rank | Ranked lower |
| Status | Admin Review [ 3 ] | In Progress [ 11605 ] |
| Status | Review [ 11605 ] | In Progress [ 3 ] |
I think this is pretty obsolete now — see also comments on DM-6349 — so I'm closing as won't fix.
| Resolution | Done [ 10000 ] | |
| Status | In Progress [ 3 ] | Won't Fix [ 10405 ] |
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.