Update app/internal_loads.py

app/internal_loads.py

@@ -962,7 +962,8 @@ def display_schedules_tab():
             for hour in range(24):
                 st.session_state[f"weekday_slider_{hour}"] = tpl["weekday"][hour]
                 st.session_state[f"weekend_slider_{hour}"] = tpl["weekend"][hour]
-        st.experimental_rerun()
+        st.session_state.module_rerun_flags["internal_loads"] = True
+        st.rerun()
 
     # ---------------------- UI FORM for name/description only ----------------------
     with st.form("schedule_form"):
@@ -982,15 +983,19 @@ def display_schedules_tab():
     with col_we:
         st.markdown("<div style='text-align:center; font-weight:bold; font-size:18px;'>Weekend</div>", unsafe_allow_html=True)
         
+    # Hide slider values, min, and max labels
     hide_elements = """
-            <style>
-                div[data-testid="stSliderTickBarMin"],
-                div[data-testid="stSliderTickBarMax"] {
-                    display: none;
-                }
-            </style>
+        <style>
+            div[data-testid="stSliderTickBarMin"],
+            div[data-testid="stSliderTickBarMax"],
+            div.stSlider > div > div > div > div[data-testid="stTickBar"] > div > div {
+                display: none;
+            }
+            div.stSlider > div > div > div > div > div > div {
+                visibility: hidden;
+            }
+        </style>
     """
-    
     st.markdown(hide_elements, unsafe_allow_html=True)
 
     weekday_values = []
@@ -1011,7 +1016,7 @@ def display_schedules_tab():
                 step=0.1,
                 value=st.session_state.get(f"weekday_slider_{hour}", 0.0),
                 label_visibility="collapsed",
-                format=None  # hides value text below slider
+                format="%.1f"
             )
             weekday_values.append(val)
 
@@ -1024,7 +1029,7 @@ def display_schedules_tab():
                 step=0.1,
                 value=st.session_state.get(f"weekend_slider_{hour}", 0.0),
                 label_visibility="collapsed",
-                format=None
+                format="%.1f"
             )
             weekend_values.append(val)
 
@@ -1040,13 +1045,18 @@ def display_schedules_tab():
                 "weekday": weekday_values,
                 "weekend": weekend_values
             }
-
+            if editor_state.get("is_edit") and editor_state.get("original_name") != name:
+                # Remove old schedule name if renamed
+                if editor_state.get("original_name") in schedules:
+                    del schedules[editor_state["original_name"]]
             st.session_state.schedule_editor = DEFAULT_STATE.copy()
             for hour in range(24):
                 st.session_state.pop(f"weekday_slider_{hour}", None)
                 st.session_state.pop(f"weekend_slider_{hour}", None)
             st.success(f"Schedule '{name}' saved successfully.")
-           
+            st.session_state.module_rerun_flags["internal_loads"] = True
+            st.rerun()
+
     # ---------------------- Show Saved ----------------------
     st.markdown("### Saved Schedules")
     if schedules:
@@ -1054,6 +1064,76 @@ def display_schedules_tab():
     else:
         st.info("No schedules saved yet.")
 
+def display_schedules_table(schedules: Dict[str, Any]):
+    """Display schedules in a table format with edit/delete buttons."""
+    if not schedules:
+        return
+    
+    # Create table data
+    table_data = []
+    for name, schedule in schedules.items():
+        weekday_peak = max(schedule.get("weekday", [0]))
+        weekend_peak = max(schedule.get("weekend", [0]))
+        
+        table_data.append({
+            "Name": name,
+            "Description": schedule.get("description", "No description"),
+            "Weekday Peak": f"{weekday_peak:.2f}",
+            "Weekend Peak": f"{weekend_peak:.2f}",
+            "Actions": name
+        })
+    
+    if table_data:
+        df = pd.DataFrame(table_data)
+        
+        # Display table
+        st.dataframe(df.drop('Actions', axis=1), use_container_width=True)
+        
+        # Display action buttons
+        st.write("**Actions:**")
+        for i, row in enumerate(table_data):
+            schedule_name = row["Actions"]
+            
+            col1, col2, col3 = st.columns([2, 1, 1])
+            
+            with col1:
+                st.write(f"{i+1}. {schedule_name}")
+            
+            with col2:
+                if st.button("Edit", key=f"edit_schedule_{schedule_name}_{i}"):
+                    # Set up editor for editing
+                    schedule_data = schedules[schedule_name]
+                    st.session_state.schedule_editor = {
+                        "is_edit": True,
+                        "original_name": schedule_name,
+                        "name": schedule_name,
+                        "description": schedule_data.get("description", ""),
+                        "template": "None",
+                        "weekday": schedule_data.get("weekday", [0.0] * 24),
+                        "weekend": schedule_data.get("weekend", [0.0] * 24)
+                    }
+                    # Initialize sliders with current values
+                    for hour in range(24):
+                        st.session_state[f"weekday_slider_{hour}"] = schedule_data.get("weekday", [0.0] * 24)[hour]
+                        st.session_state[f"weekend_slider_{hour}"] = schedule_data.get("weekend", [0.0] * 24)[hour]
+                    st.session_state.module_rerun_flags["internal_loads"] = True
+                    st.rerun()
+            
+            with col3:
+                # Check if schedule is in use or a default template before allowing deletion
+                if schedule_name not in DEFAULT_SCHEDULE_TEMPLATES:
+                    if is_schedule_in_use(schedule_name):
+                        st.write("(In Use)")
+                    else:
+                        if st.button("Delete", key=f"delete_schedule_{schedule_name}_{i}"):
+                            # Delete schedule
+                            del schedules[schedule_name]
+                            st.success(f"Schedule '{schedule_name}' deleted!")
+                            st.session_state.module_rerun_flags["internal_loads"] = True
+                            st.rerun()
+                else:
+                    st.write("(Default)")
+
 def is_schedule_in_use(schedule_name: str) -> bool:
     """
     Check if a schedule is in use by any people, lighting, equipment, or ventilation/infiltration systems.