Skip to content

DM-48936: Implement mock-based unit tests for mid-level measurement drivers #1145

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
enourbakhsh wants to merge 2 commits into
base: main
Choose a base branch
from
Open

DM-48936: Implement mock-based unit tests for mid-level measurement drivers #1145

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
8ba255a
Add unit tests for measurement drivers
enourbakhsh Jun 29, 2025
9a2052a
Preserve aperture corrections when building multi-band exposures
enourbakhsh Jul 23, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 9 additions & 7 deletions python/lsst/pipe/tasks/measurementDriver.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1139,9 +1139,9 @@ def _ensureValidInputs(
processed by this multi-band driver.
bands :
List of bands associated with the exposures in ``mExposure`` after
potential adjustments. If not provided in the input, it will be set
to a list containing only the provided (or inferred as "unknown")
``refBand`` value.
potential adjustments. If not provided for single-band input when
using this multi-band driver, it will be set to a list containing
only the provided (or inferred as "unknown") ``refBand`` value.
"""

# Perform basic checks that are shared with all driver tasks.
Expand Down Expand Up @@ -1375,12 +1375,14 @@ def _buildMultibandExposure(
# function's signature, even when using a single band.
mExposure = afwImage.MultibandExposure.fromExposures(bands, [mExposureData])

# Attach the WCS from each input exposure to the corresponding band of
# the multi-band exposure; otherwise, their WCS will be None,
# potentially causing issues downstream. Note that afwImage does not do
# this when constructing a MultibandExposure from exposures.
# Attach the WCS and ApCorrMap from each input exposure to the
# corresponding band of the multi-band exposure; otherwise, their WCS
# and ApCorrMap will be None, potentially causing issues downstream.
# Note that afwImage does not handle this when constructing a
# MultibandExposure from exposures.
for band, exposure in zip(bands, mExposureData):
mExposure[band].setWcs(exposure.getWcs())
mExposure[band].setApCorrMap(exposure.getInfo().getApCorrMap())

return mExposure

Expand Down
Loading