threeML / hawc_hal

HAWC Accelerated Likelihood - python-only framework for HAWC data analysis
BSD 3-Clause "New" or "Revised" License
11 stars 22 forks source link

No interpolation for instrument response containing a single dec bin #31

Closed henrikef closed 3 years ago

henrikef commented 5 years ago

HAL tries to interpolate between dec bins when finding the detector response to convolve a point source with. However, sometimes we will provide a detector response file containing only one dec bin (e.g. for testing, or for specialized analyses). In that case, HAL should never try to interpolate, as the interpolation routine assumes there are at least two dec bins and crashes when provided with only one dec bin. This pull request fixes that problem.

henrikef commented 4 years ago

Needs work for extended sources.

henrikef commented 4 years ago

Once the tests pass, I will merge this branch. @maloneka verified that this feature works for point sources. This is needed for development work on HAWC.