SP-3336: NB 203-series on the skymap - #213
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
| @@ -0,0 +1,738 @@ | |||
| { | |||
There was a problem hiding this comment.
Please add assert tap_service is not None to be consistent with the text in the cell above.
Reply via ReviewNB
There was a problem hiding this comment.
I think we've decided that these asserts are no longer needed.
| @@ -0,0 +1,738 @@ | |||
| { | |||
There was a problem hiding this comment.
There was a problem hiding this comment.
Thanks! I always forget to add those!
| @@ -0,0 +1,738 @@ | |||
| { | |||
There was a problem hiding this comment.
| @@ -0,0 +1,738 @@ | |||
| { | |||
There was a problem hiding this comment.
It would be helpful to include a quick demonstration showing how to access cell boundaries, similar to the tract and patch examples in the previous sections.
Reply via ReviewNB
There was a problem hiding this comment.
Good idea -- done!
galaxyumi
left a comment
galaxyumi
left a comment
There was a problem hiding this comment.
Thanks Jeff for this nice tutorial! I left some minor and major comments.
galaxyumi
left a comment
galaxyumi
left a comment
There was a problem hiding this comment.
Thanks for creating a new tutorial!
2 participants
No description provided.