Thanks, Jacek, for your comments. I've pushed the following changes to the draft branch on GitHub. Since the whole document is about to be revised, I don't propose to submit this for review but just mark it as done.
I'd suggest to reword the abstract line
I'd capitalize "object catalog"
"Object catalog" not capitalized. We currently don't capitalize consistently throughout the document (we should), and I don't see a compelling reason why this particular usage should be one way or the other.
"through an interface" - I'd change to plural, we will have multiple interfaces: sql, restful, maybe adql etc.
"Level 3 products will be generated by the users of LSST". I'd also mention that sometimes they will be brought from other surveys (e.g., they are not always "generated by lsst users")
Reworded by borrowing some text from the DPDD.
I believe this is the only place where Solar Systems Pipeline is mentioned. It is not on the L1 or L2 list of pipelines. If this is the same as MOPS, let's clarify somewhere, if not, then it is worth introducing/listing
Changed reference to Solar System Object Pipeline to DayMOPS.
I'd change this to "Process Raw Exposures to Calibrated Exposure*s*". This will require changing the diagram too.
No change made:
- Multiple snaps go into a calibrated exposure;
- The UML use case diagram will likely be removed in future rewrites: I don't intend to expend effort on it now.
extra space between "pre-" and "commissioning"
"will be heavy reliance" - did you mean "with heavily reliance"?
I believe the existing text is correct, so haven't changed it.
Do we still want to call it "prototype code" given we have been in construction for some time and are building the real thing?
Changed "prototype" to "work-in-progress".
Why is "Determination" capitalized here?
I have no idea. I changed it.
will builds a histogram --> will build a histogram
missing closing paranthesis
Capitalize "d" in "design"
The "Baseline design" capitalization seems to be in use throughout the document, but it's inconsistent with other usage. I changed it.
lsst/ap in now in lsst-dm/legacy-ap