Added explanation for "chip" parameter in Subarray notebook

tddesjardins · Dec 3, 2018 · a2e4518 · a2e4518
1 parent 4e6a589
commit a2e4518
Showing 1 changed file with 22 additions and 27 deletions.
diff --git a/acs_subarrays/acs_subarrays.ipynb b/acs_subarrays/acs_subarrays.ipynb
@@ -78,6 +78,13 @@
     "from acstools import (acs_destripe_plus, utils_calib)"
    ]
   },
+  {
+   "cell_type": "markdown",
+   "metadata": {},
+   "source": [
+    "Here we set environment variables for later use with the Calibration Reference Data System (CRDS)."
+   ]
+  },
   {
    "cell_type": "code",
    "execution_count": null,
@@ -104,13 +111,8 @@
     "\n",
     "Our data for the first example comes from a routine calibration program of two images using the post-Cycle 24 WFC1A-1K 1024 x 2072 subarray. We will only use the data associated with the observation id `JD5702JWQ`.\n",
     "\n",
-    "Using the python package `astroquery`, we can download files from the [MAST](http://archive.stsci.edu) archive."
-   ]
-  },
-  {
-   "cell_type": "markdown",
-   "metadata": {},
-   "source": [
+    "Using the python package `astroquery`, we can download files from the [MAST](http://archive.stsci.edu) archive.\n",
+    "\n",
     "<div class=\"alert alert-danger\">\n",
     "<b>MAY CHANGE:</b> The argument \"mrp_only\" stands for \"minimum recommended products only\". It currently needs to be set to False, although in the future, False is intended to be set as the default and can be left out.\n",
     "</div>"
@@ -213,7 +215,7 @@
    "source": [
     "### File Information\n",
     "\n",
-    "For more information, refer to Section <a class=\"divlink\" href=\"http://www.stsci.edu/hst/acs/documents/handbooks/currentDHB/acs_Ch23.html\">Section 2.2</a> of the ACS Data Handbook.\n",
+    "The structure of the fits files from ACS may be different depending on what kind of observation was made. For more information, refer to Section <a class=\"divlink\" href=\"http://www.stsci.edu/hst/acs/documents/handbooks/currentDHB/acs_Ch23.html\">Section 2.2</a> of the ACS Data Handbook.\n",
     "\n",
     "#### Raw Files\n",
     "\n",
@@ -236,6 +238,13 @@
     "|<b>1</b>| UDL (Image)      | (ImageHDU)   | Raw image data.                                        |"
    ]
   },
+  {
+   "cell_type": "markdown",
+   "metadata": {},
+   "source": [
+    "You can always use `.info()` on an HDUlist for an overview of the structure"
+   ]
+  },
   {
    "cell_type": "code",
    "execution_count": null,
@@ -261,7 +270,7 @@
     "\n",
     "We can call the [Calibration Reference Data System](http://www.stsci.edu/hst/observatory/crds) (CRDS) to get the associated calibration files for this image.\n",
     "\n",
-    "Next we will need to turn on the CTE correction switch in the primary header. Turning on this switch will notify the [CRDS](https://hst-crds.stsci.edu) `bestrefs` tool to add the names of the CTE correction parameters table '`PCTETAB`' and CTE-corrected dark current image '`DRKCFILE`' reference files to the header."
+    "First we will need to turn on the CTE correction switch in the primary header. Turning on this switch will notify the [CRDS](https://hst-crds.stsci.edu) `bestrefs` tool to add the names of the CTE correction parameters table '`PCTETAB`' and CTE-corrected dark current image '`DRKCFILE`' reference files to the header."
    ]
   },
   {
@@ -282,7 +291,7 @@
    "source": [
     "#### Download Calibration Files\n",
     "\n",
-    "The executable program to download calibration files is the [Calibration Reference Data System](http://www.stsci.edu/hst/observatory/crds) (CRDS). It can be run from the terminal to download the files specific to your observation. This program is packaged with astroconda. \n",
+    "The CRDS program can be run from the terminal to download the files specific to your observation. This program is packaged with astroconda. \n",
     "\n",
     "The command line input to download our files for `jd5702jwq_raw.fits` is as follows:\n",
     "\n",
@@ -369,7 +378,7 @@
    "source": [
     "#### Access OSCN Table\n",
     "\n",
-    "The name in the header is of the format ref\\\\$oscn_name.fits, therefore we need to split the string on the '$' character."
+    "The name in the header is of the format '<font face='courier'>ref\\\\$oscn_name.fits</font>', therefore we need to split the string on the '$' character."
    ]
   },
   {
@@ -420,13 +429,6 @@
     "oscntable = Table(oscnrec)"
    ]
   },
-  {
-   "cell_type": "markdown",
-   "metadata": {},
-   "source": [
-    "<font color=\"red\">Check the output here. Do we already know the raw image's dimensions at this point?</font>"
-   ]
-  },
   {
    "cell_type": "markdown",
    "metadata": {},
@@ -546,7 +548,7 @@
    "cell_type": "markdown",
    "metadata": {},
    "source": [
-    "<font color=\"red\">Do we have an explanation for the following line?</font>"
+    "The following line sets `chip` to 1 if the subarray is on WFC1, and 2 if the subarray is on WFC2."
    ]
   },
   {
@@ -704,14 +706,7 @@
    "cell_type": "markdown",
    "metadata": {},
    "source": [
-    "Using the same check we did earlier, we can use `check_oscntab` to see if our settings are defined. If everything was done correctly, the following line should print \"Success\"!"
-   ]
-  },
-  {
-   "cell_type": "markdown",
-   "metadata": {},
-   "source": [
-    "<font color=\"red\">Check output here.</font>"
+    "Using the same check we did earlier, we can use `check_oscntab` to see if our settings are defined. If everything was done correctly, the following line should print \"True\"!"
    ]
   },
   {