Merge pull request #66 from AllenNeuralDynamics/han_fix_32

Fix: collapsing slider range when only one sessions is filtered
AllenNeuralDynamics · Apr 15, 2024 · e1ccfc7 · e1ccfc7
2 parents f1704e0 + cf698c0
commit e1ccfc7
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/code/Home.py b/code/Home.py
@@ -12,7 +12,7 @@
 
 """
 
-__ver__ = 'v2.2.1'
+__ver__ = 'v2.2.2'
 
 import pandas as pd
 import streamlit as st

diff --git a/code/util/streamlit.py b/code/util/streamlit.py
@@ -288,9 +288,9 @@ def filter_dataframe(df: pd.DataFrame,
                     else:
                         x = df[column]               
 
-                    _min = float(x.min())
-                    _max = float(x.max())
-                    step = (_max - _min) / 100
+                    _min = float(x.min() - 0.05 * abs(x.min()))  # Avoid collapsing the range
+                    _max = float(x.max() + 0.05 * abs(x.max()))
+                    step = (_max - _min) / min(100, len(np.unique(x)) + 1) # Avoid too many steps
 
                     c_hist = st.container()  # Histogram
 
@@ -306,10 +306,12 @@ def filter_dataframe(df: pd.DataFrame,
                                         zip(st.session_state[f'filter_{column}'], (_min, _max))]
                     else:
                         default_value = (_min, _max)
-
+
+                    default_value = list(default_value)
+                    default_value[0] = max(_min, default_value[0])
+                    default_value[1] = min(_max, default_value[1])
                     st.session_state[f'filter_{column}'] = default_value
 
-
                     user_num_input = st.slider(
                         f"Values for {column}",
                         label_visibility='collapsed',