Skip to content

suppress adding (combining) of cal-spectra on fits file level#457

Merged
nicHoch merged 1 commit intoi4Ds:masterfrom
nicHoch:cal_energy_add_fix
Mar 10, 2026
Merged

suppress adding (combining) of cal-spectra on fits file level#457
nicHoch merged 1 commit intoi4Ds:masterfrom
nicHoch:cal_energy_add_fix

Conversation

@nicHoch
Copy link
Collaborator

@nicHoch nicHoch commented Mar 10, 2026

No description provided.

@nicHoch nicHoch requested a review from samaloney March 10, 2026 09:31
# we have to check in the for loop as some products might create
# multiple files and some of them might be combinable and some not
except NotCombineException as nc:
logger.warning(nc)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do the email alerts trigger on warning or just info?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no just on error

Copy link
Collaborator

@samaloney samaloney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nicHoch nicHoch merged commit 01a7086 into i4Ds:master Mar 10, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants