Revert "testing loop-less methodology to reduce O notation"

This reverts commit 26ca5bf.

restoring main to be be up to date with upstream
:wq

jdaviz/configs/cubeviz/helper.py

@@ -22,9 +22,6 @@ class Cubeviz(ImageConfigHelper, LineListMixin):
     _default_flux_viewer_reference_name = "flux-viewer"
     _default_image_viewer_reference_name = "image-viewer"
 
-    _loaded_flux_cube = None
-    _loaded_uncert_cube = None
-
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
         self.app.hub.subscribe(self, AddDataMessage,

jdaviz/configs/cubeviz/plugins/parsers.py

@@ -252,14 +252,12 @@ def _parse_hdulist(app, hdulist, file_name=None,
 
         elif data_type == 'uncert':
             app.add_data_to_viewer(uncert_viewer_reference_name, data_label)
-            app._jdaviz_helper._loaded_uncert_cube = app.data_collection[data_label]
 
         else:  # flux
             # Add flux to top left image viewer
             app.add_data_to_viewer(flux_viewer_reference_name, data_label)
             # Add flux to spectrum viewer
             app.add_data_to_viewer(spectrum_viewer_reference_name, data_label)
-            app._jdaviz_helper._loaded_flux_cube = app.data_collection[data_label]
 
 
 def _parse_jwst_s3d(app, hdulist, data_label, ext='SCI',
@@ -309,12 +307,6 @@ def _parse_jwst_s3d(app, hdulist, data_label, ext='SCI',
     if viewer_name == flux_viewer_reference_name:
         app.add_data_to_viewer(spectrum_viewer_reference_name, data_label)
 
-    if data_type == 'flux':
-        print(f"this is the data label : {data_label}")
-        app._jdaviz_helper._loaded_flux_cube = app.data_collection[data_label]
-    elif data_type == 'uncert':
-        app._jdaviz_helper._loaded_uncert_cube = app.data_collection[data_label]
-
 
 def _parse_esa_s3d(app, hdulist, data_label, ext='DATA', flux_viewer_reference_name=None,
                    spectrum_viewer_reference_name=None):
@@ -361,11 +353,6 @@ def _parse_esa_s3d(app, hdulist, data_label, ext='DATA', flux_viewer_reference_n
     app.add_data_to_viewer(flux_viewer_reference_name, data_label)
     app.add_data_to_viewer(spectrum_viewer_reference_name, data_label)
 
-    if data_type == 'flux':
-        app._jdaviz_helper._loaded_flux_cube = app.data_collection[data_label]
-    if data_type == 'uncert':
-        app._jdaviz_helper._loaded_uncert_cube = app.data_collection[data_label]
-
 
 def _parse_spectrum1d_3d(app, file_obj, data_label=None,
                          flux_viewer_reference_name=None, spectrum_viewer_reference_name=None,
@@ -412,15 +399,9 @@ def _parse_spectrum1d_3d(app, file_obj, data_label=None,
 
         if attr == 'flux':
             app.add_data_to_viewer(flux_viewer_reference_name, cur_data_label)
-            app._jdaviz_helper._loaded_flux_cube = app.data_collection[cur_data_label]
-            #print(f"data label: {data_label}")
-            #print(f'flux cube stuff: {app._jdaviz_helper._flux_cube_data}')
+            app.add_data_to_viewer(spectrum_viewer_reference_name, cur_data_label)
         elif attr == 'uncertainty':
             app.add_data_to_viewer(uncert_viewer_reference_name, cur_data_label)
-            app._jdaviz_helper._loaded_uncert_cube = app.data_collection[cur_data_label]
-
-            #app._jdaviz_helper._uncert_cube_data = data_label
-            #print(f'uncert cube stuff: {app._jdaviz_helper._uncert_cube_data}')
         # We no longer auto-populate the mask cube into a viewer
 
 
@@ -465,10 +446,8 @@ def _parse_ndarray(app, file_obj, data_label=None, data_type=None,
     if data_type == 'flux':
         app.add_data_to_viewer(flux_viewer_reference_name, data_label)
         app.add_data_to_viewer(spectrum_viewer_reference_name, data_label)
-        app._jdaviz_helper._loaded_flux_cube = app.data_collection[data_label]
     elif data_type == 'uncert':
         app.add_data_to_viewer(uncert_viewer_reference_name, data_label)
-        app._jdaviz_helper._loaded_uncert_cube = app.data_collection[data_label]
 
 
 def _parse_gif(app, file_obj, data_label=None, flux_viewer_reference_name=None,

jdaviz/configs/cubeviz/plugins/spectral_extraction/spectral_extraction.py

@@ -108,7 +108,6 @@ def collapse_to_spectrum(self, add_data=True, **kwargs):
             Examples include ``propagate_uncertainties`` and ``operation_ignores_mask``.
         """
         # get glue Data objects for the spectral cube and uncertainties
-        '''
         flux_viewer = self._app.get_viewer(
             self._app._jdaviz_helper._default_flux_viewer_reference_name
         )
@@ -117,17 +116,7 @@ def collapse_to_spectrum(self, add_data=True, **kwargs):
         )
         [spectral_cube] = flux_viewer.data()
         [uncert_cube] = uncert_viewer.data()
-        '''
-        flux_data = self._app._jdaviz_helper._loaded_flux_cube
-        uncert_data = self._app._jdaviz_helper._loaded_uncert_cube
-
-        print(flux_data)
-        print("\n\n")
-        print(uncert_data)
-        if flux_data != None and uncert_data != None:
-            spectral_cube = flux_data
-            uncert_cube = uncert_data
-
+
         # This plugin collapses over the *spatial axes* (optionally over a spatial subset,
         # defaults to ``No Subset``). Since the Cubeviz parser puts the fluxes
         # and uncertainties in different glue Data objects, we translate the spectral