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

Remove catalog and data browser Flask web apps #1103

merged 3 commits into from Aug 16, 2017


Copy link

@cdeil cdeil commented Aug 16, 2017

This pull request removes the catalog and data browser Flask web apps.

Those were things I started to add two or three years ago, because I wanted the functionality (to browse catalog data and FITS data from HESS) and wanted to learn how Python-powered web pages with Flask work. Looking back at this now, it's pretty clear that Flask web apps is not the way to go in Gammapy. If someone wants to build web apps as GUIs to do analysis with Gammapy, they can in their projects / repos, but given that Gammapy developers aren't familar / excited by HTML / Javascript / Jinja / Flask we shouldn't put and maintain it in the Gammapy package.

I do think we should allow widgets or web GUIs in Gammapy as long as they are almost exclusively Python code and thus not very hard to maintain and potentially useful to a good fraction of users. Concretely Bokeh is something to look at, and in the coming weeks JupyterLab will be released, which is the evolution of the Jupyter notebook and Jupyter widgets that we already use and that many devs and users are familiar with and like (see e.g. or the upcoming and ).

So if anyone is interested to implement Jupyter widgets or other GUI functionality (or just example docs) how to browse gamma-ray data or catalog data, please open a new issue if you have any questions or a pull request if you have implemented something.

For catalog data there's also by now for cases where you just want to look up some info on a given source.

@cdeil cdeil added the cleanup label Aug 16, 2017
@cdeil cdeil added this to the 0.7 milestone Aug 16, 2017
@cdeil cdeil self-assigned this Aug 16, 2017
@cdeil cdeil merged commit 3cd3ae9 into gammapy:master Aug 16, 2017
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
continuous-integration/appveyor/pr AppVeyor build succeeded
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

1 participant