refactor(dashboard_app): make more explicit max_length for smart_join()

alphanome-ai · Dec 28, 2023 · 0d9ac75 · 0d9ac75
1 parent 0877edf
commit 0d9ac75
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/dev_utils/dashboard_app/python_utils.py b/dev_utils/dashboard_app/python_utils.py
@@ -1,4 +1,4 @@
-def smart_join(strings, max_length=50):
+def smart_join(strings, max_length: int):
     result = ""
     for i, string in enumerate(strings):
         # Add a newline if the string is long or the previous string was long

diff --git a/dev_utils/dashboard_app/select_reports.py b/dev_utils/dashboard_app/select_reports.py
@@ -78,7 +78,7 @@ def __call__(self, value):
 
     to_add = [k for k in addlist if k not in current]
     current.extend(to_add)
-    queries = smart_join(current)
+    queries = smart_join(current, max_length=50)
 
     st.session_state.select_reports__queries = queries
     st_utils.st_unkeep("select_reports__queries")