# DM-27013 broke ap_verify runs

XMLWordPrintable

#### Details

• Type: Bug
• Status: Done
• Resolution: Done
• Fix Version/s: None
• Component/s:
• Labels:
None
• Story Points:
1
• Sprint:
AP S22-5 (April)
• Team:
• Urgent?:
No

#### Description

Krzysztof Findeisen reported that my merge of DM-27013 broke the ap_verify runs for at least DECam, and possibly HSC. I've verified this in my own run (see below). This should be a simple fix to the pipeline config.

  File "/home/parejkoj/lsst/lsstsw/stack/lsst-scipipe-3.0.0/Linux64/meas_algorithms/gf36ae6ace1+09a7694010/python/lsst/meas/algorithms/loadReferenceObjects.py", line 228, in validate  raise pexConfig.FieldValidationError(LoadReferenceObjectsConfig.anyFilterMapsToThis, lsst.pex.config.config.FieldValidationError: Field 'astromRefObjLoader.anyFilterMapsToThis' failed validation: filterMap and anyFilterMapsToThis are mutually exclusive For more information see the Field definition at:  File meas/algorithms/loadReferenceObjects.py:199 (LoadReferenceObjectsConfig) and the Config definition at:  File meas/algorithms/loadIndexedReferenceObjects.py:34 () 

#### Activity

Hide
Krzysztof Findeisen added a comment - - edited

It's both DECam and HSC. All three datasets use filterMap to spell out the filters used by the dataset's built-in refcats; I think they predate anyFilterMapsToThis entirely.

Show
Krzysztof Findeisen added a comment - - edited It's both DECam and HSC. All three datasets use filterMap to spell out the filters used by the dataset's built-in refcats; I think they predate anyFilterMapsToThis entirely.
Hide
John Parejko added a comment -

Krzysztof Findeisen: do you mind taking a look at this? Three PRs, two of them are the same changes, one I had to do some other cleanups to have a place to make the change.

Show
John Parejko added a comment - Krzysztof Findeisen : do you mind taking a look at this? Three PRs, two of them are the same changes, one I had to do some other cleanups to have a place to make the change. https://github.com/lsst/ap_verify_ci_hits2015/pull/38 https://github.com/lsst/ap_verify_hits2015/pull/43 https://github.com/lsst/ap_verify_ci_cosmos_pdr2/pull/26
Hide
John Parejko added a comment -

Thanks for the review: I cleaned up the gen2 cruft in the hits datasets too. Hopefully this all works in production now.

Show
John Parejko added a comment - Thanks for the review: I cleaned up the gen2 cruft in the hits datasets too. Hopefully this all works in production now.

#### People

Assignee:
John Parejko
Reporter:
John Parejko
Reviewers:
Krzysztof Findeisen
Watchers:
John Parejko, Krzysztof Findeisen, Tim Jenness