astropy / astropy-v5.0-paper

Astropy Paper III: To be published with the next LTS release of astropy core (v5.0, Fall 2021)
9 stars 38 forks source link

Add first version of CTA section #28

Closed maxnoe closed 2 years ago

maxnoe commented 2 years ago

@adonath, please have a look

adrn commented 2 years ago

This is great @maxnoe and @adonath and is exactly what we were looking for -- thanks!

@maxnoe is going to share this with CTA folks for a review, and then we can do a final review and merge.

nmearl commented 2 years ago

@maxnoe any update on the CTA review -- is this ready for a final review and merge?

maxnoe commented 2 years ago

@nmearl unfortunately not yet, I asked for a review but did not yet receive an answer.

RichardWhite109 commented 2 years ago

@maxnoe I've now taken a pass on behalf of SAPO, some very small changes in the attached. Let me know if anything is unclear d220208_gammapy.txt

maxnoe commented 2 years ago

@RichardWhite109 thank you, I agree with all the changes and just implemented them.

@nmearl I think this can be merged then

adonath commented 2 years ago

Thanks @RichardWhite109 and @maxnoe!