Skip to content
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.

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

Add energy reference for exposure map #1100

merged 1 commit into from Aug 25, 2017


Copy link

@JouvinLea JouvinLea commented Aug 8, 2017

Add an argument to compute the exposure at another energy than the log center of the energy band of the map. For example if we want the flux at 1 TeV, we want to be able to compute the exposure at a reference energy of 1 TeV.

Copy link

@JouvinLea JouvinLea commented Aug 23, 2017

@adonath @cdeil
Can we merge it? it's a small change

@cdeil cdeil self-assigned this Aug 25, 2017
@cdeil cdeil added the feature label Aug 25, 2017
@cdeil cdeil added this to the 0.7 milestone Aug 25, 2017
Copy link

@cdeil cdeil commented Aug 25, 2017

@JouvinLea - Thanks!

Looks good to me. Merging.
@adonath - If you prefer some other way to implement this, please open a follow-up PR.

@cdeil cdeil merged commit c13b173 into gammapy:master Aug 25, 2017
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
continuous-integration/travis-ci/pr The Travis CI build passed
@cdeil cdeil changed the title add energy reference for exposure map Add energy reference for exposure map Aug 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants