LSSTScienceCollaborations / ObservingStrategy

A community white paper about LSST observing strategy, with quantifications via the the Metric Analysis Framework.
59 stars 75 forks source link

Comments from Nelson Padilla (SAC) on weak lensing section of Cosmology chapter #630

Open michaelstrauss opened 7 years ago

michaelstrauss commented 7 years ago

First of all, this is an extremely carefully written section, I found it very clear and thorough and therefore I only have a few comments about explaining further some details, perhaps illustrating metrics with simple figures, and making some more quantitative estimates to back the weak lensing requirements.

drphilmarshall commented 7 years ago

Thanks Nelson (via Michael)!

@jmeyers314 Would you mind taking a few minutes to work through Nelson's comments on the WL section and submit a quick PR with a revised version? Thank you! :-)

MichelleLochner commented 7 years ago

@jmeyers314 how's this going? Consider this a friendly prod from chapter editor. :)

jmeyers314 commented 7 years ago

Thanks @MichelleLochner -- I'll try to have a look at this tomorrow or early next week. :)

drphilmarshall commented 7 years ago

Thanks Josh! I've pasted in Nelson's remaining feedback into the comments of the tex file, to be considered during v2 development, and will label accordingly. The overlap with WFIRST in the early years is something that needs more work in general.

jmeyers314 commented 7 years ago

Hey Phil, I just realized that it seems like I haven't actually merged my fork of this back into origin. What's the procedure for doing that?

jmeyers314 commented 7 years ago

Think I figured it out. There's now a PR waiting for somebody to approve/merge.

drphilmarshall commented 7 years ago

Thanks Josh - ill take a look while preparing the arxiv submission!