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 data management tools #352

Merged
merged 1 commit into from Oct 11, 2015

Conversation

Projects
None yet
2 participants
@cdeil
Member

cdeil commented Sep 9, 2015

This PR does some things (should have splitted in several PRs, but it was just too tempting to keep working in this one branch):

  • Add gammapy-data-show as a quick-look command line tool to check data and IRF files. It replaces examples/plot_irfs.py and gammapy/scripts/irf_info.py.
  • Add gammapy-data-browser, a Flask web app to quickly browse data / IRF files.
  • Fix some IRF plots
  • Fix test fails that came up because Astropy quantity is now more strict when comparing to non-quantities.

The gammapy-data-browser is very much unfinished.
I'm only merging this PR now because I want what's here in the 0.4 release and don't have more time.
Will continue working on the IRF classes and the data show and browse tools in future PRs.

@cdeil cdeil added bug cleanup labels Sep 9, 2015

@cdeil cdeil self-assigned this Sep 9, 2015

@cdeil cdeil added this to the 0.4 milestone Sep 9, 2015

@cdeil cdeil force-pushed the cdeil:plot-irfs branch from 3c93b47 to 404af72 Sep 9, 2015

@cdeil cdeil changed the title from Fix IRF plots to Add data browser web app and fix some IRF plots Sep 11, 2015

@cdeil cdeil force-pushed the cdeil:plot-irfs branch from 03a1656 to 4ef6611 Sep 14, 2015

@cdeil cdeil changed the title from Add data browser web app and fix some IRF plots to Add data management tools Sep 16, 2015

@cdeil cdeil force-pushed the cdeil:plot-irfs branch from 6d6171f to 16d3683 Sep 16, 2015

@joleroi

This comment has been minimized.

Contributor

joleroi commented Sep 18, 2015

I merged [ #351]. It would be nice to have the updated plotting routinges so feel free to rebase&merge

@cdeil cdeil referenced this pull request Sep 18, 2015

Closed

test fails in master #355

@cdeil cdeil force-pushed the cdeil:plot-irfs branch 2 times, most recently from 599dd08 to 332fff9 Sep 26, 2015

@cdeil cdeil force-pushed the cdeil:plot-irfs branch 2 times, most recently from a3799cc to 513d927 Oct 4, 2015

@cdeil cdeil force-pushed the cdeil:plot-irfs branch from cef878c to 41016e9 Oct 11, 2015

cdeil added a commit that referenced this pull request Oct 11, 2015

Merge pull request #352 from cdeil/plot-irfs
Add data management tools

@cdeil cdeil merged commit 6446a30 into gammapy:master Oct 11, 2015

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@cdeil cdeil referenced this pull request Oct 12, 2015

Closed

Setup Tests Fail #366

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