Repository navigation
DM-56280: add Prompt 102_Schedule_preview series (Rubin Schedule View… - #219
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
| @@ -0,0 +1,688 @@ | |||
| { | |||
There was a problem hiding this comment.
The description of the LSSTCam visits database is good to keep as is, but the description of the other schedule viewer tutorial could be moved to the "related tutorials" section at the end.
Related tutorials: The other 100-level Schedule Viewer tutorial covers the service's more advanced query options (predicate filtering, column selection, and alternate response formats) and an analysis of a retrieved schedule. The 100-level commissioning tutorial on the LSSTCam visits database.
Reply via ReviewNB
There was a problem hiding this comment.
thanks. i've moved this to the end
| @@ -0,0 +1,688 @@ | |||
| { | |||
There was a problem hiding this comment.
There was a problem hiding this comment.
it seems that the documentation site was never published, so I just replaced with the github URL
| @@ -0,0 +1,688 @@ | |||
| { | |||
There was a problem hiding this comment.
| @@ -0,0 +1,697 @@ | |||
| { | |||
There was a problem hiding this comment.
| @@ -0,0 +1,697 @@ | |||
| { | |||
There was a problem hiding this comment.
I would suggest keeping these as separate steps (different cells) and perhaps moving the local_noon/night_utc calculation to section 2.
Reply via ReviewNB
There was a problem hiding this comment.
thank you. done
| @@ -0,0 +1,697 @@ | |||
| { | |||
There was a problem hiding this comment.
Maybe print the date used as well, rather than "that night" since someone might change the date so it'd be nice to get that feedback. Same suggestion for section 5.
Reply via ReviewNB
There was a problem hiding this comment.
good idea. done!
| @@ -0,0 +1,697 @@ | |||
| { | |||
There was a problem hiding this comment.
GloriaFA
left a comment
GloriaFA
left a comment
There was a problem hiding this comment.
Reviewed both notebooks and added comments. One more thing, the schedule viewer map (https://usdf-rsp.slac.stanford.edu/obsloctap/skymap) links to the old tutorial in the top right. It might be good to make sure that link still works after migrating the tutorials in the documentation.
Approving since my comments were small :)
2b0279a to
61fdecc
Compare
|
thank you Gloria, once published, i'll open a new PR on /lsst-dm/obsloctap to update the link in the schedule viewer map. |
2 participants
…er queries)
Migrating the Rubin Schedule Viewer tutorial out of Commissioning/ and into Prompt/102_Schedule_preview/. Now split into two notebooks.
102.1 — Query the Rubin Schedule Viewer: the existing content (GUI, ObsLocTAP background, key columns, basic forecast and past-schedule queries).
102.2 — Advanced queries: new content covering the service's column selection, predicate filtering, and alternate response formats (CSV/VOTable/Parquet), plus a worked analysis of a night's schedule, and links to the prenight/multiprenight/nightsum scheduler reports.