Merge pull request #154 from ojustino/notebook-img-parsing

Update image handling in notebooks

Author: ojustino

CHANGES.rst

@@ -19,10 +19,10 @@ API Changes
 
 - All operations now accept Spectrum1D and Quantity-type images. All
   accepted image types are now processed internally as Spectrum1D objects
-  [#144]
+  [#144, #154]
 
 - All operations' ``image`` attributes are now coerced Spectrum1D
-  objects [#144]
+  objects [#144, #154]
 
 Bug Fixes
 ^^^^^^^^^

notebook_sandbox/jwst_boxcar/boxcar_extraction.ipynb

@@ -90,7 +90,7 @@
    "source": [
     "# use a jwst datamodel to provide a good interface to the data and wcs info\n",
     "s2d = datamodels.open(s2dfile)\n",
-    "image = s2d.slits[0].data"
+    "image = np.array(s2d.slits[0].data)"
    ]
   },
   {
@@ -361,7 +361,7 @@
    ],
    "source": [
     "plt.figure(figsize=(15, 15))\n",
-    "plt.imshow(bg.bkg_wimage[::,0:100], origin=\"lower\")\n",
+    "plt.imshow(bg.bkg_wimage[::,0:100].data, origin=\"lower\")\n",
     "plt.title(\"slit[0] slice\")"
    ]
   },
@@ -395,7 +395,7 @@
    ],
    "source": [
     "plt.figure(figsize=(15, 15))\n",
-    "plt.imshow(bg.bkg_image(image)[::,0:100], norm=norm_data, origin=\"lower\")\n",
+    "plt.imshow(bg.bkg_image(image)[::,0:100].data, norm=norm_data, origin=\"lower\")\n",
     "plt.title(\"slit[0] slice\")"
    ]
   },
@@ -429,7 +429,7 @@
    ],
    "source": [
     "plt.figure(figsize=(15, 15))\n",
-    "plt.imshow(bg.sub_image(image)[::,0:100], norm=norm_data, origin=\"lower\")\n",
+    "plt.imshow(bg.sub_image(image)[::,0:100].data, norm=norm_data, origin=\"lower\")\n",
     "plt.title(\"slit[0] slice\")"
    ]
   },
@@ -471,7 +471,7 @@
    "source": [
     "bg_med = Background.two_sided(image, ext_center, bkg_sep, width=bkg_width, statistic='median')\n",
     "plt.figure(figsize=(15, 15))\n",
-    "plt.imshow(bg_med.bkg_wimage[::,0:100], origin=\"lower\")\n",
+    "plt.imshow(bg_med.bkg_wimage[::,0:100].data, origin=\"lower\")\n",
     "plt.title(\"slit[0] slice\")"
    ]
   },
@@ -548,7 +548,7 @@
     "## About this notebook\n",
     "\n",
     "**Author:** Ivo Busko, JWST\n",
-    "**Updated On:** 2022-11-11"
+    "**Updated On:** 2022-12-05"
    ]
   },
   {
@@ -565,13 +565,6 @@
     "[Top of Page](#top)\n",
     "<img style=\"float: right;\" src=\"https://raw.githubusercontent.com/spacetelescope/notebooks/master/assets/stsci_pri_combo_mark_horizonal_white_bkgd.png\" alt=\"Space Telescope Logo\" width=\"200px\"/> "
    ]
-  },
-  {
-   "cell_type": "code",
-   "execution_count": null,
-   "metadata": {},
-   "outputs": [],
-   "source": []
   }
  ],
  "metadata": {

notebook_sandbox/tracing_options.ipynb

@@ -71,7 +71,7 @@
    "source": [
     "# use a jwst datamodel to provide a good interface to the data and wcs info\n",
     "s2d = datamodels.open(s2dfile)\n",
-    "image = s2d.slits[0].data\n",
+    "image = np.array(s2d.slits[0].data)\n",
     "norm_data = simple_norm(image, \"sqrt\")"
    ]
   },
@@ -154,7 +154,7 @@
    ],
    "source": [
     "plt.figure(figsize=(15, 15))\n",
-    "plt.imshow(bg.sub_image(image), norm=norm_data, aspect=5, origin=\"lower\")\n",
+    "plt.imshow(bg.sub_image(image).data, norm=norm_data, aspect=5, origin=\"lower\")\n",
     "plt.plot(auto_trace_gauss.trace, color='r')\n",
     "plt.title(\"slit[0] slice\")"
    ]
@@ -338,7 +338,7 @@
     "## About this notebook\n",
     "\n",
     "**Author:** Kyle Conroy, JWST\n",
-    "**Updated On:** 2022-11-16"
+    "**Updated On:** 2022-12-05"
    ]
   },
   {
@@ -348,13 +348,6 @@
     "[Top of Page](#top)\n",
     "<img style=\"float: right;\" src=\"https://raw.githubusercontent.com/spacetelescope/notebooks/master/assets/stsci_pri_combo_mark_horizonal_white_bkgd.png\" alt=\"Space Telescope Logo\" width=\"200px\"/> "
    ]
-  },
-  {
-   "cell_type": "code",
-   "execution_count": null,
-   "metadata": {},
-   "outputs": [],
-   "source": []
   }
  ],
  "metadata": {