diff --git a/data_preprocessor.py b/data_preprocessor.py index d800b9f..d6f531d 100644 --- a/data_preprocessor.py +++ b/data_preprocessor.py @@ -197,18 +197,19 @@ class DataPreprocessor: Returns: Tuple of (normalized_priority, priority_level) - priority_level: "very_high", "high", "medium", "low", "monitoring", "complete", None + priority_level: "critical", "high", "medium", "low", None + Note: "monitoring" and "complete" are statuses, not priorities, so they return None """ if not priority: return "", None priority_lower = priority.lower() - # Very High priority - if '(1) very high' in priority_lower or '(1) very hgh' in priority_lower: - return priority, "very_high" - if 'very high' in priority_lower or 'very hgh' in priority_lower: - return priority, "very_high" + # Critical priority (formerly Very High) + if '(1) critical' in priority_lower or '(1) very high' in priority_lower or '(1) very hgh' in priority_lower: + return priority, "critical" + if 'critical' in priority_lower or 'very high' in priority_lower or 'very hgh' in priority_lower: + return priority, "critical" # High priority (but not Very High) if '(2) high' in priority_lower or '(2) hgh' in priority_lower: @@ -230,13 +231,8 @@ class DataPreprocessor: if priority_lower == 'low': return priority, "low" - # Monitoring priority - if '(5) monitoring' in priority_lower: - return priority, "monitoring" - - # Complete priority - if '(6) complete' in priority_lower: - return priority, "complete" + # Note: "(5) monitoring" and "(6) complete" are statuses, not priorities + # If they appear in the priority column, they will return None (no priority) return priority, None @@ -517,7 +513,7 @@ class DataPreprocessor: 'closed': 0, 'open': 0, 'monitor': 0, - 'very_high': [], + 'critical': [], 'high': [], 'unaddressed': [], 'recent_added': [], @@ -536,8 +532,8 @@ class DataPreprocessor: else: vendors[vendor]['open'] += 1 - if item['priority_level'] == 'very_high': - vendors[vendor]['very_high'].append(item) + if item['priority_level'] == 'critical': + vendors[vendor]['critical'].append(item) elif item['priority_level'] == 'high': vendors[vendor]['high'].append(item) @@ -608,9 +604,9 @@ class DataPreprocessor: output_lines.append("") # Priority items - if vendor_data['very_high']: - output_lines.append(f"VERY HIGH PRIORITY ({len(vendor_data['very_high'])} items):") - for item in vendor_data['very_high']: + if vendor_data['critical']: + output_lines.append(f"CRITICAL PRIORITY ({len(vendor_data['critical'])} items):") + for item in vendor_data['critical']: output_lines.append(f" {item['punchlist_name']} | {item['status']} | {item['date_identified_str']}") output_lines.append("") diff --git a/html_generator.py b/html_generator.py index dabe626..d096145 100644 --- a/html_generator.py +++ b/html_generator.py @@ -51,7 +51,7 @@ def get_priority_badge_class(priority: Optional[str]) -> str: if not priority: return "badge-secondary" priority_lower = priority.lower() - if "very high" in priority_lower or "(1)" in priority_lower: + if "critical" in priority_lower or "very high" in priority_lower or "(1)" in priority_lower: return "badge-critical" elif "high" in priority_lower or "(2)" in priority_lower: return "badge-high" @@ -740,7 +740,7 @@ def generate_html_content(report_data: Dict) -> str: - +
@@ -620,7 +620,7 @@