Update app/data_validation.py

app/data_validation.py

@@ -263,17 +263,13 @@ class DataValidation:
                 messages.append(f"People Load #{i+1}: Name is required.")
             
             # Check numeric fields
-            if person.get("quantity", 0) < 0:
+            if person.get("num_people", 0) < 0:
                 is_valid = False
-                messages.append(f"People Load #{i+1}: Quantity must be non-negative.")
+                messages.append(f"People Load #{i+1}: Number of people must be non-negative.")
             
-            if person.get("sensible_heat", 0) < 0:
+            if person.get("hours_in_operation", 0) <= 0:
                 is_valid = False
-                messages.append(f"People Load #{i+1}: Sensible heat must be non-negative.")
-            
-            if person.get("latent_heat", 0) < 0:
-                is_valid = False
-                messages.append(f"People Load #{i+1}: Latent heat must be non-negative.")
+                messages.append(f"People Load #{i+1}: Hours in operation must be positive.")
         
         # Check lighting loads
         lighting = internal_loads.get("lighting", [])
@@ -291,6 +287,10 @@ class DataValidation:
             if light.get("usage_factor", 0) < 0 or light.get("usage_factor", 0) > 1:
                 is_valid = False
                 messages.append(f"Lighting Load #{i+1}: Usage factor must be between 0 and 1.")
+            
+            if light.get("hours_in_operation", 0) <= 0:
+                is_valid = False
+                messages.append(f"Lighting Load #{i+1}: Hours in operation must be positive.")
         
         # Check equipment loads
         equipment = internal_loads.get("equipment", [])
@@ -312,6 +312,10 @@ class DataValidation:
             if equip.get("radiation_fraction", 0) < 0 or equip.get("radiation_fraction", 0) > 1:
                 is_valid = False
                 messages.append(f"Equipment Load #{i+1}: Radiation fraction must be between 0 and 1.")
+            
+            if equip.get("hours_in_operation", 0) <= 0:
+                is_valid = False
+                messages.append(f"Equipment Load #{i+1}: Hours in operation must be positive.")
         
         return is_valid, messages
     
@@ -417,7 +421,7 @@ class DataValidation:
             Boolean indicating whether validation succeeded
         """
         if data_key not in session_state:
-            st.error(f"No {data_key.replace('_', ' ')} data found.")
+            st.error(f"No {data_key.replace('_', ' ').title()} data found.")
             return False
         
         # Validate data
@@ -487,4 +491,4 @@ if __name__ == "__main__":
             data_validation.validate_building_info,
             "building_info",
             "Building information is valid!"
-        )
+        )