Details
-
Type:
Bug
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: ap_verify
-
Labels:
-
Story Points:3
-
Epic Link:
-
Sprint:AP S21-4 (March)
-
Team:Alert Production
-
Urgent?:No
Description
While updating the ap_verify_hits2015 repo on DM-28438, I got the following error:
convertRepo WARN: Skipping ingestion for '/tmp/tmpnw_qx4on/calibingested/cpFLAT/2015-02-23/Y/FLAT-2015-02-23.fits': Could not fetch record for required dimension physical_filter via keys {'instrument': 'DECam', 'detector': 62, 'physical_filter': 'Y'}.
|
This looks suspiciously related to the ongoing filter conversion. Investigate and fix.
Attachments
Issue Links
- relates to
-
- Done
Activity
The culprit is obs.base.gen2to3.BandToPhysicalFilterKeyHandler (and higher-level code) considering only the bands and physical filters of any filter definition. Unfortunately, the way the existing code works looks like it would be very hard to generalize without unexpected side effects, so we decided on #dm-middleware-dev to just add some special-case code to obs_decam.
Krzysztof Findeisen
added a comment - The culprit is obs.base.gen2to3.BandToPhysicalFilterKeyHandler (and higher-level code) considering only the bands and physical filters of any filter definition. Unfortunately, the way the existing code works looks like it would be very hard to generalize without unexpected side effects, so we decided on #dm-middleware-dev to just add some special-case code to obs_decam . In addition to the code changes in obs_decam, I've re-converted ap_verify_hits2015 (to make the Y-band files available in Gen 3) and ap_verify_ci_hits2015 (to include the post-DM-23985 linearizer files as curated calibs).
Krzysztof Findeisen
added a comment - In addition to the code changes in obs_decam , I've re-converted ap_verify_hits2015 (to make the Y-band files available in Gen 3) and ap_verify_ci_hits2015 (to include the post- DM-23985 linearizer files as curated calibs). Thanks for clearly explaining the situation and for fixing it!
'Y' is not a DECam physical filter. "Y DECam c0005 10095.0 1130.0" is, however. Looks like this is one of the few cases where an alias from the FilterDefinitions was actually used!