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 SkyMap class #487

Merged
merged 8 commits into from Mar 11, 2016

Conversation

Projects
None yet
2 participants
@adonath
Member

adonath commented Mar 10, 2016

This PR introduces a SkyMap class as basic data container for all image/map based analysis in gammapy. In subsequent PRs, I will adapt existing code to make use of it.

@cdeil cdeil added this to the 0.4 milestone Mar 11, 2016

@cdeil

This comment has been minimized.

Show comment
Hide comment
@cdeil

cdeil Mar 11, 2016

Member

What about things like the standalone coordinates function?
Maybe remove it and go more all-in on SkyMap?

Member

cdeil commented Mar 11, 2016

What about things like the standalone coordinates function?
Maybe remove it and go more all-in on SkyMap?

adonath added a commit that referenced this pull request Mar 11, 2016

@adonath adonath merged commit f8f5fa9 into gammapy:master Mar 11, 2016

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details

@adonath adonath referenced this pull request Mar 11, 2016

Merged

Improve SkyMap #488

@cdeil cdeil changed the title from Introduce SkyMap class to Add SkyMap class Apr 20, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment