Skip to content

Fix crash for background observations in get_short_band_name #91

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

Merged
thomaswilliamsastro merged 1 commit into from
Aug 18, 2025
Merged

Fix crash for background observations in get_short_band_name #91

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
1 change: 1 addition & 0 deletions CHANGES.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
1.2.1 (Unreleased)
==================

- Fix crash for background observations in ``get_short_band_name``
- For background association, strip whitespace and lowercase for more robust matching
- Imports update - now supports numpy>2.0!
- Keep track of expected stripe direction in ``multi_tile_destripe_step``
Expand Down
3 changes: 2 additions & 1 deletion pjpipe/utils/utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -241,14 +241,15 @@ def get_band_ext(band):

return band_ext


def get_short_band_name(band):
"""Get a stripped down short name for a band"""

band_short = copy.deepcopy(band)

# Strip background
if "_bgr" in band_short:
band_short = band_short.replace("_bgr")
band_short = band_short.replace("_bgr", "")

# Strip instrument names
if "_niriss" in band_short:
Expand Down