From 32bb09e92285e54e0bdf1c2c6de9091bf9125ed7 Mon Sep 17 00:00:00 2001 From: Kyle Conroy Date: Mon, 23 Sep 2024 09:13:17 -0400 Subject: [PATCH] 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 --- CHANGES.rst | 2 ++ jdaviz/configs/default/plugins/model_fitting/model_fitting.py | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/CHANGES.rst b/CHANGES.rst index 145c8f02c9..b371412871 100644 --- 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 index 3ed30d20f6..4213baca47 100644 --- 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()