LSSTScienceCollaborations / StackClub

Learning the LSST software Stack, by writing jupyter notebook tutorials.
https://stackclub.readthedocs.io/
MIT License
45 stars 17 forks source link

Cleaning up, trying to write about _mapper files #167

Closed drphilmarshall closed 5 years ago

drphilmarshall commented 5 years ago

Hey @rmorgan10 - nice work with the sky coverage plotting! I figured it was time to clean up the bottom ends of our two notebooks so we can look at them together. Because PR #159 is still live, and your work is being usefully reviewed there by @kadrlica and @SimonKrughoff , I thought I'd work in a patch branch and offer you these changes to help respond to that review. I think you can merge this when you are ready, without comment from Alex or Simon. Then, once you have merged this PR, my changes will appear in #159 , and then we can continue discussion of Alex's comments and overall design, there. Sound OK?