Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@fjebaker
Copy link
Owner

No description provided.

fjebaker added 10 commits May 12, 2024 14:21
There was an issue with one of the old EXOSAT datasets where there was a
mismatch in the channels in the response matrix and the spectrum, which
lead to the masking selecting the wrong bins when fitting.

This commit adds a check (on by default, but toggleable) that drops
channels in the response matrix that are not in the spectrum.

I've made it optional, as if someone is interested in seeing the full
response matrix they can use the same API but just pass false to the
keyword argument.
@fjebaker fjebaker merged commit e1899a0 into main May 12, 2024
@fjebaker fjebaker deleted the fergus/goodness branch May 12, 2024 15:55
@fjebaker fjebaker mentioned this pull request May 12, 2024
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