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

Misc cleanup #225

Merged
merged 9 commits into from Jan 6, 2015

Conversation

Projects
None yet
1 participant
@cdeil
Member

cdeil commented Jan 5, 2015

This is a large changeset, but it's mostly trivial cleanup stuff:

  • Get rid of imageutils (has been merged in Astropy core)
  • Always use code-block directive in docs
  • Put plot scripts in docs in separate Python files (easier to re-use and debug)
  • Declare dependencies with version numbers in setup.py
  • Remove license line from examples because it's just destracting for end users
  • Move examples/significance_study.ipynb file to gammapy-extra
  • Add GoodTimeInterval class stub because it was referenced in the docs
  • Fix some typos and PEP8

@cdeil cdeil added the cleanup label Jan 6, 2015

@cdeil cdeil added this to the 0.2 milestone Jan 6, 2015

@cdeil cdeil self-assigned this Jan 6, 2015

@cdeil

This comment has been minimized.

Show comment
Hide comment
@cdeil

cdeil Jan 6, 2015

Member

Merging this now ... I'll make a new issue for the unrelated travis-ci tests (tests abort on Python 3, probably due to a segfault in numpy).

Member

cdeil commented Jan 6, 2015

Merging this now ... I'll make a new issue for the unrelated travis-ci tests (tests abort on Python 3, probably due to a segfault in numpy).

cdeil added a commit that referenced this pull request Jan 6, 2015

@cdeil cdeil merged commit 39caf3b into gammapy:master Jan 6, 2015

0 of 2 checks passed

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

@cdeil cdeil changed the title from Some cleanup to Misc cleanup Apr 8, 2015

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