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

Finish change to Click in gammapy.scripts #1240

merged 9 commits into from Jan 2, 2018


None yet
1 participant

cdeil commented Dec 28, 2017

This PR finishes the change in gammapy.scripts to use Click that was started in #1235 .

It adds an extensive docs page for gammapy.scripts, including reference documentation for the existing commands using . I've added a lot of information on the docs page how it works, that is mostly targeted at contributors, not end users. I think this is OK, since at the top there is a big fat warning that the CLI doesn't really offer much useful functionality now for end users. I'm hoping that this information will be useful to someone or a small group of people to take on the challenge to develop the Gammapy high-level interface in 2018, and then once something nice is in place, the documentation will very likely be reorganised into parts that are useful for users and technical info for developers.

This PR leaves gammapy as the only command line tool, removing a few others like gammapy-image-model-sherpa or gammapy-data-select. I hesitated to remove the data select to do run selection based on sky location, time or other parameters, but in the end did remove it because it probaby should be re-written in a better way, e.g. using astropy-regions and taking ds9 region strings for more flexible / powerful / simple sky region selection, i.e. the CLI interface and implementation would change. Also, it's still in the git version history if someone wants to have a look at the old implementation as an example or starting point to inform a new better one.

@cdeil cdeil added the cleanup label Dec 28, 2017

@cdeil cdeil added this to the 0.7 milestone Dec 28, 2017

@cdeil cdeil self-assigned this Dec 28, 2017

@cdeil cdeil added the docs label Jan 2, 2018

@cdeil cdeil merged commit a597a4c into gammapy:master Jan 2, 2018

0 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
continuous-integration/travis-ci/pr The Travis CI build is in progress

@cdeil cdeil referenced this pull request May 3, 2018


Document scripts in docs #24

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