✅ Add production speed check to product readiness validation

- Add speed_data to DemandFilter class initialization
- Load speed data in load_data() method alongside other required data
- Add production speed check to is_product_ready_for_optimization()
- Products without speed data are now excluded from optimization
- Remove redundant speed data loading and warning in filter_products()
- Speed check is now part of the core readiness criteria
- Prevents optimization errors from missing speed data
- Improves data validation feedback with 'Missing production speed data' reason

src/demand_filtering.py

@@ -25,6 +25,7 @@ class DemandFilter:
         self.kit_dependencies = None
         self.line_assignments = None
         self.team_requirements = None
+        self.speed_data = None
         
     def load_data(self, force_reload=False):
         """Load all necessary data for filtering"""
@@ -79,6 +80,10 @@ class DemandFilter:
                 'Humanizer': kits_df.set_index('Kit')['Humanizer'].to_dict()
             }
             
+            # Load production speed data
+            self.speed_data = extract.read_package_speed_data()
+            
+            print(f"✅ Filtering data loaded: {len(self.demand_data)} products with demand, {len(self.speed_data)} with speed data")
             return True
             
         except Exception as e:
@@ -123,6 +128,7 @@ class DemandFilter:
         1) Should be right type - standalone master, subkit, prepack
         2) Should have line assignment
         3) Should have staffing requirements
+        4) Should have production speed data
         
         Returns:
             Tuple[bool, List[str]]: (is_ready, exclusion_reasons)
@@ -159,6 +165,10 @@ class DemandFilter:
         if total_staff == 0:
             exclusion_reasons.append("Zero staffing requirements")
         
+        # Check production speed data
+        if self.speed_data is None or product_id not in self.speed_data:
+            exclusion_reasons.append("Missing production speed data")
+        
         is_ready = len(exclusion_reasons) == 0
         return is_ready, exclusion_reasons
     
@@ -225,24 +235,10 @@ class DemandFilter:
             for reason, count in reason_counts.items():
                 print(f"  • {reason}: {count} products")
         
-        # Print data quality warnings for included products (without recursion)
-        # Load speed data for validation
-        speed_data = None
-        try:
-            from src.config import optimization_config
-            from src.preprocess import extract
-            speed_data = extract.read_package_speed_data()
-        except Exception as e:
-            print(f"Warning: Could not load speed data for validation: {e}")
-        
-        if speed_data:
-            included_without_speed = sum(1 for pid in included_products if pid not in speed_data)
-            if included_without_speed > 0:
-                print(f"\n⚠️  DATA QUALITY WARNING: {included_without_speed} included products missing speed data")
-        
+        # Print data quality warnings for included products
         included_without_hierarchy = sum(1 for pid in included_products if self.standalone_master_filter(pid)[0] == "unclassified")
         if included_without_hierarchy > 0:
-            print(f"⚠️  DATA QUALITY WARNING: {included_without_hierarchy} included products missing hierarchy data")
+            print(f"\n⚠️  DATA QUALITY WARNING: {included_without_hierarchy} included products missing hierarchy data")
         
         return included_products, included_demand, excluded_products, excluded_demand