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 tutorials/astropy_introduction.ipynb #2118

Merged
merged 1 commit into from Apr 17, 2019
Merged

Remove tutorials/astropy_introduction.ipynb #2118

merged 1 commit into from Apr 17, 2019

Conversation

@cdeil
Copy link
Member

@cdeil cdeil commented Apr 17, 2019

@adonath - How do you feel about the Astropy introduction tutorial?

https://github.com/gammapy/gammapy/blob/master/tutorials/astropy_introduction.ipynb
https://docs.gammapy.org/0.11/notebooks/astropy_introduction.html

Maybe remove it and instead just refer to this one that's mostly the same content, but more polished?
https://github.com/Asterics2020-Obelics/School2019/blob/master/astropy/astropy_hands_on.ipynb

If you want to keep tutorials/astropy_introduction.ipynb, OK, then a minimal edit would be to update https://docs.gammapy.org/0.11/tutorials.html#basics to refer to the latest version of https://github.com/Asterics2020-Obelics/School2019/blob/master/astropy/astropy_hands_on.ipynb

@cdeil cdeil added the question label Apr 17, 2019
@cdeil cdeil added this to the 0.12 milestone Apr 17, 2019
@cdeil cdeil added this to To Do in DOCUMENTATION via automation Apr 17, 2019
@adonath
Copy link
Member

@adonath adonath commented Apr 17, 2019

@cdeil I'm fine to remove it...

@cdeil cdeil assigned cdeil and unassigned adonath Apr 17, 2019
@cdeil cdeil requested a review from adonath Apr 17, 2019
@cdeil
Copy link
Member Author

@cdeil cdeil commented Apr 17, 2019

@adonath - Please review.

I just left a link to astropy.org and the tutorial from the ASTERICS school, both of which contain links to other learning resources for Astropy. Didn't think it's useful to maintain more references here, but can add some if you want.

Copy link
Member

@adonath adonath left a comment

Thanks, @cdeil! No comments from my side...

@adonath adonath merged commit dd47308 into gammapy:master Apr 17, 2019
8 of 9 checks passed
DOCUMENTATION automation moved this from To Do to Done Apr 17, 2019
@adonath adonath changed the title Remove tutorials/astropy_introduction.ipynb ? Remove tutorials/astropy_introduction.ipynb May 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
DOCUMENTATION
  
Done
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants