Rmap tickets for JWST build 10.0 #956
Conversation
| 'filetype' : 'INTERPIXEL CAPACITANCE', | ||
| 'instrument' : 'MIRI', | ||
| 'mapping' : 'REFERENCE', | ||
| 'name' : 'miri_ipc.rmap', |
There was a problem hiding this comment.
Not positive about the names I'm giving them, since other rmaps tend to list out full examples like
"'name' : 'jwst_nirspec_wavelengthrange_0000.rmap'," for nirspec_wavelengthrange.rmap,
meanwhile others are simplified like the ones I provided, as in:
"'name' : 'nircam_superbias.rmap'," for nircam_superbias.rmap.
There was a problem hiding this comment.
At this level, the name is not really relevant. More generic is preferred only because these are just for validation, and the name field is not involved. For a solid rule, it should be the name of the the actual file itself.
| @@ -8,8 +8,8 @@ header = { | |||
| 'mapping' : 'REFERENCE', | |||
| 'name' : 'jwst_miri_apcorr.rmap', | |||
| 'observatory' : 'JWST', | |||
| 'parkey' : (('META.EXPOSURE.TYPE',), ('META.OBSERVATION.DATE', 'META.OBSERVATION.TIME')), | |||
| 'sha1sum' : 'e2255db2e45b01b85aa4686d6e1811ad77ff33b8', | |||
| 'parkey' : (('META.EXPOSURE.TYPE', 'META.INSTRUMENT.CHANNEL'), ('META.OBSERVATION.DATE', 'META.OBSERVATION.TIME')), | |||
There was a problem hiding this comment.
I feel like the parkey tuple needs to have it's second setting match, so not sure this is in the right place or if I need a third tuple for just META.INSTRUMENT parkeys.
This PR contains changes for CCD-1360, 1361 and 1362.