search

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

fixed misleading error message #27

Closed cbrisboi closed 4 years ago

cbrisboi commented 5 years ago

This branch is to fix #26. active_measurements only looks at the maptree, so saying the missing bin is in the response file leads the user in the wrong direction for investigating a problem

henrikef commented 5 years ago

Do you want to fix the typo (it -> is) as well while you're at it?

henrikef commented 5 years ago

Tests are failing due to issues with root/pyroot. I'm inclined to accept the pull request anyway since it's such a small change.

cbrisboi commented 5 years ago

I can confirm it works, but since it is a small change any potential problems caused by it should be easily traceable and fixable.

henrikef commented 4 years ago

Going to merge this since only error messages have been changed.