The "filter" should only be set for raw ccds and not for raw amps in obs_lsst

Description

With https://rubinobs.atlassian.net/browse/DM-21048#icft=DM-21048 it is possible to read in raw data even when the filter is not listed in the obs package, although a warning is given. This can be annoying because currently a warning is given for each amplifier, although it is unnecessary to set/check the filter when reading in amps, it only needs to be done (and is done!) when assembling the ccd.

It appears as though the place to put the change is in https://github.com/lsst/obs_lsst/blob/aeaebe9ad92137ac7a57e2bd580746e2cbffe16f/python/lsst/obs/lsst/lsstCamMapper.py#L393 where the call to self._standardizeExposure should have filter=False.

Checklist

Lucidchart Diagrams

Issue Matrix

hide

Activity

Show:
Merlin Fisher-Levine
October 18, 2019 at 3:34 PM

Looks good. Funny that that functionality was already in obs_base and didn't need adding, just needed using, but I won't question it!

Eli Rykoff
October 18, 2019 at 3:31 PM

This is a one-line change.

Done
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Reviewers

Merlin Fisher-Levine

Story Points

RubinTeam

Components

Checklist

Created October 17, 2019 at 2:08 PM
Updated November 9, 2019 at 2:11 AM
Resolved October 18, 2019 at 3:50 PM