fixes post review

Author: MelissaGraham

Commissioning/101_lsstcam_visits_database.ipynb

@@ -1050,7 +1050,6 @@
    "outputs": [],
    "source": [
     "query = \"observation_reason.str.contains('ddf')\"\n",
-    "# df_ddf = df.query(query).groupby(['target_name', 'band']).agg({'seq_num': 'count'})\n",
     "df_ddf = df.query(query).groupby(['observation_reason', 'band']).agg({'seq_num': 'count'})\n",
     "df_ddf.rename({'seq_num': 'count'}, axis=1, inplace=True)\n",
     "df_ddf = df_ddf.reset_index('band').pivot(columns=[\"band\"]).droplevel(0, axis=1)\n",

Commissioning/102_rubin_schedule_viewer.ipynb

@@ -88,30 +88,33 @@
     "The ObsLocTAP data model (table schema) for the scheduled observations is defined in the `ivoa.obsplan` table -- <a href=\"https://www.ivoa.net/documents/ObsLocTAP/20210609/PR-ObsLocTAP-1.0-20210609.pdf\">Observation Locator \n",
     "Table Access Protocol Version 1.0</a>\n",
     "\n",
-    "The database contains 28 columns in total, but these are the key columns.\n",
+    "The database contains 28 columns in total, and these are the key columns.\n",
     "\n",
     "* `t_min` : The start time (MJD).\n",
     "* `t_max` : The stop time (MJD).\n",
-    "* `s_ra` : The Right Ascension of the visit or planned observation (degrees).\n",
-    "* `s_dec` : The Declination of the visit or planned observation (degrees).\n",
-    "* `s_region` : Sky region covered by the data product (expressed in ICRS frame).\n",
+    "* `s_ra` : The Right Ascension of the visit or planned observation (deg).\n",
+    "* `s_dec` : The Declination of the visit or planned observation (deg).\n",
     "* `em_min` : The filter minimum wavelength (m).\n",
     "* `em_max` : The filter maximum wavelength (m).\n",
     "* `target_name` : The name of the astronomical object observed, if any.\n",
     "* `tracking_type` : One of the following values: Sidereal, Solar-system-object-tracking, Fixed-az-el-transit.\n",
     "* `priority` : The Priority level { 0, 1, 2}. 0 = in the target queue as the next visit\n",
     "(meaning its execution is highly likely), 1 = in the 1 hour look-ahead summit queue (meaning likely),\n",
     "2 = 24 hour pre-night simulation, meaning it is not in the summit queue and has a high degree of uncertainty. \n",
-    "* `execution_status` : Execution status of the prediction: One of the following values: Planned, Scheduled, Unscheduled, Performed, Aborted.\n",
-    "* `obs_id` : The observation ID from the Camera.\n",
     "* `t_planning` : Time (MJD) when this observation has been added or modified into the planning log.\n",
+    "* `execution_status` : Execution status of the prediction: One of the following values: Planned, Scheduled, Unscheduled, Performed, Aborted.\n",
+    "\n",
+    "Key columns that remained yet unpopulated when this tutorial was last updated.\n",
+    "\n",
+    "* `s_region` : Sky region covered by the data product (expressed in ICRS frame).\n",
+    "* `obs_id` : The observation ID from the Camera (currently the same as `target_name`).\n",
     "\n",
-    "These are also columns of interest, but which are the same for most observations taken with LSSTCam.\n",
+    "These are also columns of interest, but are generally the same for most observations taken with LSSTCam.\n",
     "\n",
-    "* `t_plan_exptime` : The planned or scheduled exposure time.\n",
-    "* `t_exptime` : The total exposure time = `t_max - t_min`.\n",
-    "* `s_fov` : The diameter (bounds) of the covered region.\n",
-    "* `s_resolution` : The spatial resolution of the data expressed as FWHM.\n",
+    "* `t_plan_exptime` : The planned or scheduled exposure time (s).\n",
+    "* `t_exptime` : The total exposure time = `t_max - t_min` (s).\n",
+    "* `s_fov` : The diameter (bounds) of the covered region (deg).\n",
+    "* `s_resolution` : The spatial resolution of the data expressed as FWHM (arcsec/pixel).\n",
     "* `facility_name` : Name of the facility used for this observation e.g Rubin Simonyi, AuxTel.\n",
     "* `instrument_name` : Name of the instrument used for this observation e.g LSSTCam, LSSTComCam.\n",
     "\n",