show unit cols by default in model fitting results table (#3196)

* show unit cols by default in model fitting results table - especially useful now that switching input units is supported
spacetelescope · Sep 23, 2024 · 32bb09e · 32bb09e
1 parent 7391556
commit 32bb09e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -23,6 +23,8 @@ New Features
 
 - New configuration for ramp/Level 1 data products from Roman WFI and JWST [#3120, #3148, #3167, #3171]
 
+- Unit columns are now visible by default in the results table in model fitting. [#3196]
+
 Cubeviz
 ^^^^^^^
 

diff --git a/jdaviz/configs/default/plugins/model_fitting/model_fitting.py b/jdaviz/configs/default/plugins/model_fitting/model_fitting.py
@@ -149,9 +149,9 @@ def __init__(self, *args, **kwargs):
 
         self.table.headers_avail = headers
         self.table.headers_visible = headers
-        # when model parameters are added as columns, only show the value columns by default
+        # when model parameters are added as columns, only show the value/unit columns by default
         # (other columns can be show in the dropdown by the user)
-        self.table._new_col_visible = lambda colname: colname.split(':')[-1] not in ('unit', 'fixed', 'uncert', 'std')  # noqa
+        self.table._new_col_visible = lambda colname: colname.split(':')[-1] not in ('fixed', 'uncert', 'std')  # noqa
 
         # set the filter on the viewer options
         self._update_viewer_filters()