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

CameraMapper uses the butler to set the exposure filter

    XMLWordPrintable

Details

    Description

      When loading a calexp, the _setFilter function in CameraMapper tries to look up the filter name from the butler, rather than using the information contained in the exposure itself. This means that to be loaded successfully, all exposures must either have their filter in the filename/path (decam does not) or be listed in the registry. This was discovered when sullivan was attempting to load decam calexps that had been generated without corresponding raw files being registered, which should have been possible.

      The docstring and the function's behavior seem to have diverged about six years ago with this commit: https://github.com/lsst/obs_base/commit/8719cf189e6b37d4e55d09c09e061ac67293bbe1

      Tagging ktl and npease on this just as an example of a butler lookups lurking in places they probably don't belong. This doesn't require a fix from their end; it's a science pipelines issue.

      Attachments

        Issue Links

          Activity

            No builds found.
            ctslater Colin Slater created issue -
            krzys Krzysztof Findeisen made changes -
            Field Original Value New Value
            Link This issue relates to DM-12672 [ DM-12672 ]
            krzys Krzysztof Findeisen made changes -
            Labels sciencepipelines filter-remake sciencepipelines
            ctslater Colin Slater made changes -
            Resolution Done [ 10000 ]
            Status To Do [ 10001 ] Invalid [ 11005 ]

            This code has been reworked on other tickets; this ticket is no longer relevant.

            ctslater Colin Slater added a comment - This code has been reworked on other tickets; this ticket is no longer relevant.

            People

              Unassigned Unassigned
              ctslater Colin Slater
              Colin Slater, Ian Sullivan, John Parejko, Kian-Tat Lim, Nate Pease [X] (Inactive), Simon Krughoff (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.