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

DM-27013 broke ap_verify runs

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ap_verify
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      AP S22-5 (April)
    • Team:
      Alert Production
    • 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 (<module>)
      

        Attachments

          Issue Links

            Activity

            Hide
            krzys 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
            krzys 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
            Parejkoj 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
            Parejkoj 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
            Parejkoj 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
            Parejkoj 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:
              Parejkoj John Parejko
              Reporter:
              Parejkoj John Parejko
              Reviewers:
              Krzysztof Findeisen
              Watchers:
              John Parejko, Krzysztof Findeisen, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.