Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
MelissaGraham
left a comment
There was a problem hiding this comment.
Header isn't right -- use the code and logo from the current template in RTN-045 repo (_static).
Shorten learning objective to one sentence.
Can remove "matplotlib" from packages line, this can be only the core packages being taught... then again the whole purpose of this tutorial is plotting so maybe leave it after all...
Credit line, we don't track every individual's name in the credit line anymore, just use developed by CST. I have the Jira record to track contributions for performance reviews.
Not that it's a wellspring of info but, could link to https://pipelines.lsst.io/modules/lsst.geom/index.html as documentation for lsst.geom. And maybe similar for lsst.afw?
Section 1.2 it says "and define font sizes for the focal plane figure" but only colormap is defined.
I said it before but: I like the exercises for the learner, very achievable and useful to work through.
Approving PR so you can merge after the above little fixes.
No description provided.