Update app.py

app.py

@@ -1159,10 +1159,9 @@ import plotly.graph_objects as go
 
 # =================== OBJECTIVE 5: Operator Fatigue Risk Gradient Dashboard =====================
 # ... (kode sebelumnya tetap sama) ...
-
 st.subheader("OBJECTIVE 5: See your team’s fatigue Fatigue Hazard Profile!")
 
-# ✅ Custom CSS — diperbarui dengan styling list untuk rekomendasi
+# Custom CSS — tetap seperti sebelumnya (sudah sesuai preferensi)
 st.markdown("""
 <style>
     .big-title {
@@ -1263,30 +1262,20 @@ st.markdown("""
     .recommendation-title {
         font-weight: bold;
         color: white;
-        margin-bottom: 12px;
+        margin-bottom: 8px;
         font-size: 14px;
         background: rgba(255,255,255,0.2);
         padding: 8px;
         border-radius: 5px;
         border-left: 4px solid white;
     }
-    .recommendation-box ul {
-        padding-left: 24px;
-        margin: 12px 0;
-        line-height: 1.6;
-        font-size: 14px;
-    }
-    .recommendation-box ul li {
-        margin-bottom: 8px;
-    }
     .recommendation-reason {
         font-size: 12px;
-        margin-top: 12px;
+        margin-top: 10px;
         padding: 8px;
-        background: rgba(255,255,255,0.15);
+        background: rgba(255,255,255,0.1);
         border-radius: 5px;
-        border-left: 3px solid rgba(255,255,255,0.4);
-        font-style: italic;
+        border-left: 3px solid rgba(255,255,255,0.3);
     }
 </style>
 """, unsafe_allow_html=True)
@@ -1459,9 +1448,7 @@ else:
         </div>
         """, unsafe_allow_html=True)
 
-        # ===============================================================
-        # PLOT FUNCTION — UPDATED: color for slope=0 is now #FFD700
-        # ===============================================================
+        # ✅ Function definition at correct indentation level
         def plot_chart(data, title):
             if data.empty:
                 fig = go.Figure()
@@ -1545,7 +1532,7 @@ else:
             st.plotly_chart(plot_chart(top_coal, "HAULING COAL Operators (Hazard Gradient)"), use_container_width=True)
 
         # ===============================================================
-        # AI INSIGHTS — tetap dalam bahasa Inggris, tanpa emoticon
+        # AI INSIGHTS — DIPERBAIKI: Risk Summary jadi 1 box + 3 list
         # ===============================================================
         col_insight1, col_insight2 = st.columns(2)
 
@@ -1557,6 +1544,7 @@ else:
                 ob_one_time = len(top_ob[top_ob['slope'] == 0])
                 ob_avg_risk = top_ob['weekly_avg'].mean()
                 ob_max_risk = top_ob['weekly_avg'].max()
+
                 ob_insights = []
                 if ob_worsening > ob_improving:
                     ob_insights.append(f"{ob_worsening} out of 10 top risk operators are showing <span class='trend-up'>worsening</span> trends.")
@@ -1564,15 +1552,20 @@ else:
                     ob_insights.append(f"{ob_improving} out of 10 top risk operators are showing <span class='trend-down'>improvement</span>.")
                 if ob_one_time > 0:
                     ob_insights.append(f"{ob_one_time} operators are classified as <b>One Time Event</b> (single-week activity).")
+                else:
+                    ob_insights.append("No operators classified as <b>One Time Event</b>.")
                 ob_insights.append(f"Average risk: {ob_avg_risk:.2f} events/week (max: {ob_max_risk:.2f}).")
 
-                for insight in ob_insights:
-                    st.markdown(f"""
-                    <div class="ai-insight-box">
-                        <div class="ai-insight-title">Risk Summary</div>
-                        <p>{insight}</p>
-                    </div>
-                    """, unsafe_allow_html=True)
+                st.markdown(f"""
+                <div class="ai-insight-box">
+                    <div class="ai-insight-title">Risk Summary</div>
+                    <ul style="padding-left: 20px; margin: 8px 0; line-height: 1.5;">
+                        <li>{ob_insights[0]}</li>
+                        <li>{ob_insights[1]}</li>
+                        <li>{ob_insights[2]}</li>
+                    </ul>
+                </div>
+                """, unsafe_allow_html=True)
             else:
                 st.info("No OB HAULER data for analysis.")
 
@@ -1584,6 +1577,7 @@ else:
                 coal_one_time = len(top_coal[top_coal['slope'] == 0])
                 coal_avg_risk = top_coal['weekly_avg'].mean()
                 coal_max_risk = top_coal['weekly_avg'].max()
+
                 coal_insights = []
                 if coal_worsening > coal_improving:
                     coal_insights.append(f"{coal_worsening} out of 10 top risk operators are showing <span class='trend-up'>worsening</span> trends.")
@@ -1591,116 +1585,92 @@ else:
                     coal_insights.append(f"{coal_improving} out of 10 top risk operators are showing <span class='trend-down'>improvement</span>.")
                 if coal_one_time > 0:
                     coal_insights.append(f"{coal_one_time} operators are classified as <b>One Time Event</b> (single-week activity).")
+                else:
+                    coal_insights.append("No operators classified as <b>One Time Event</b>.")
                 coal_insights.append(f"Average risk: {coal_avg_risk:.2f} events/week (max: {coal_max_risk:.2f}).")
 
-                for insight in coal_insights:
-                    st.markdown(f"""
-                    <div class="ai-insight-box">
-                        <div class="ai-insight-title">Risk Summary</div>
-                        <p>{insight}</p>
-                    </div>
-                    """, unsafe_allow_html=True)
+                st.markdown(f"""
+                <div class="ai-insight-box">
+                    <div class="ai-insight-title">Risk Summary</div>
+                    <ul style="padding-left: 20px; margin: 8px 0; line-height: 1.5;">
+                        <li>{coal_insights[0]}</li>
+                        <li>{coal_insights[1]}</li>
+                        <li>{coal_insights[2]}</li>
+                    </ul>
+                </div>
+                """, unsafe_allow_html=True)
             else:
                 st.info("No HAULING COAL data for analysis.")
 
         # ===============================================================
-        # RECOMMENDATIONS — FORMAT BARU: Judul + 3 List Langsung
+        # RECOMMENDATIONS — TIDAK DIUBAH SAMA SEKALI
         # ===============================================================
         col_rec1, col_rec2 = st.columns(2)
 
-        # === OB HAULER RECOMMENDATIONS (3-point list) ===
-        with col_rec1:
+        def generate_recommendations(top_ob, top_coal):
+            rec = {}
             if not top_ob.empty:
                 w = len(top_ob[top_ob['slope'] > 0])
                 ot = len(top_ob[top_ob['slope'] == 0])
                 avg = top_ob['weekly_avg'].mean()
-
-                rec_list_ob = []
-
-                # Point 1: Trend-driven action
                 if w > 5:
-                    rec_list_ob.append("Conduct targeted fatigue risk assessments for operators with worsening trends (slope > 0).")
+                    r = "Prioritize fatigue intervention for operators with worsening trends."
+                    reason = "High proportion of deteriorating operators signals emerging fatigue risks."
                 elif ot > 4:
-                    rec_list_ob.append("Investigate data completeness — high count of One Time Events may reflect inconsistent reporting.")
+                    r = "Validate data completeness — high One Time Event count may indicate reporting gaps."
+                    reason = "Operators with single-week data cannot yield reliable trend analysis."
                 elif avg > 8:
-                    rec_list_ob.append("Review shift scheduling and rest-break compliance to reduce event frequency.")
+                    r = "Review scheduling and rest protocols to reduce event frequency."
+                    reason = "Elevated average event rate increases cumulative fatigue exposure."
                 else:
-                    rec_list_ob.append("Continue current protocols with routine monitoring of top-risk operators.")
+                    r = "Maintain current protocols with targeted monitoring."
+                    reason = "Risk profile is stable; focus on sustaining safe practices."
+                rec['ob'] = r
+                rec['ob_reason'] = reason
 
-                # Point 2: One-Time Event follow-up
-                if ot > 0:
-                    rec_list_ob.append(f"Re-engage {ot} operators flagged as <b>One Time Event</b> to verify activity continuity.")
+            if not top_coal.empty:
+                w = len(top_coal[top_coal['slope'] > 0])
+                ot = len(top_coal[top_coal['slope'] == 0])
+                avg = top_coal['weekly_avg'].mean()
+                if w > 5:
+                    r = "Prioritize fatigue intervention for operators with worsening trends."
+                    reason = "High proportion of deteriorating operators signals emerging fatigue risks."
+                elif ot > 4:
+                    r = "Validate data completeness — high One Time Event count may indicate reporting gaps."
+                    reason = "Operators with single-week data cannot yield reliable trend analysis."
+                elif avg > 8:
+                    r = "Review scheduling and rest protocols to reduce event frequency."
+                    reason = "Elevated average event rate increases cumulative fatigue exposure."
                 else:
-                    rec_list_ob.append("No One Time Event operators identified — trend analysis remains reliable.")
-
-                # Point 3: Benchmarking
-                rec_list_ob.append(f"Use cohort average ({avg:.2f} events/week) as baseline for monthly fatigue KPI reviews.")
+                    r = "Maintain current protocols with targeted monitoring."
+                    reason = "Risk profile is stable; focus on sustaining safe practices."
+                rec['coal'] = r
+                rec['coal_reason'] = reason
+            return rec
 
-                # Ensure exactly 3 items
-                while len(rec_list_ob) < 3:
-                    rec_list_ob.append("—")
+        ai_rec = generate_recommendations(top_ob, top_coal)
 
+        with col_rec1:
+            if 'ob' in ai_rec:
                 st.markdown("### OB HAULER Recommendations")
                 st.markdown(f"""
                 <div class="recommendation-box">
                     <div class="recommendation-title">Action Plan</div>
-                    <ul>
-                        <li>{rec_list_ob[0]}</li>
-                        <li>{rec_list_ob[1]}</li>
-                        <li>{rec_list_ob[2]}</li>
-                    </ul>
-                    <div class="recommendation-reason">
-                        Based on trend slope, activity duration, and cohort event frequency.
-                    </div>
+                    <div>{ai_rec['ob']}</div>
+                    <div class="recommendation-reason">AI Reasoning: {ai_rec['ob_reason']}</div>
                 </div>
                 """, unsafe_allow_html=True)
             else:
                 st.info("No OB HAULER recommendations.")
 
-        # === HAULING COAL RECOMMENDATIONS (3-point list) ===
         with col_rec2:
-            if not top_coal.empty:
-                w = len(top_coal[top_coal['slope'] > 0])
-                ot = len(top_coal[top_coal['slope'] == 0])
-                avg = top_coal['weekly_avg'].mean()
-
-                rec_list_coal = []
-
-                # Point 1: Trend-driven action
-                if w > 5:
-                    rec_list_coal.append("Conduct targeted fatigue risk assessments for operators with worsening trends (slope > 0).")
-                elif ot > 4:
-                    rec_list_coal.append("Investigate data completeness — high count of One Time Events may reflect inconsistent reporting.")
-                elif avg > 8:
-                    rec_list_coal.append("Review shift scheduling and rest-break compliance to reduce event frequency.")
-                else:
-                    rec_list_coal.append("Continue current protocols with routine monitoring of top-risk operators.")
-
-                # Point 2: One-Time Event follow-up
-                if ot > 0:
-                    rec_list_coal.append(f"Re-engage {ot} operators flagged as <b>One Time Event</b> to verify activity continuity.")
-                else:
-                    rec_list_coal.append("No One Time Event operators identified — trend analysis remains reliable.")
-
-                # Point 3: Benchmarking
-                rec_list_coal.append(f"Use cohort average ({avg:.2f} events/week) as baseline for monthly fatigue KPI reviews.")
-
-                # Ensure exactly 3 items
-                while len(rec_list_coal) < 3:
-                    rec_list_coal.append("—")
-
+            if 'coal' in ai_rec:
                 st.markdown("### HAULING COAL Recommendations")
                 st.markdown(f"""
                 <div class="recommendation-box">
                     <div class="recommendation-title">Action Plan</div>
-                    <ul>
-                        <li>{rec_list_coal[0]}</li>
-                        <li>{rec_list_coal[1]}</li>
-                        <li>{rec_list_coal[2]}</li>
-                    </ul>
-                    <div class="recommendation-reason">
-                        Based on trend slope, activity duration, and cohort event frequency.
-                    </div>
+                    <div>{ai_rec['coal']}</div>
+                    <div class="recommendation-reason">AI Reasoning: {ai_rec['coal_reason']}</div>
                 </div>
                 """, unsafe_allow_html=True)
             else:
@@ -1708,8 +1678,8 @@ else:
 
     except Exception as e:
         st.error(f"Error in Top 10 Operator analysis: {str(e)}")
-        # st.exception(e)  # Uncomment during development only
-# =================== OBJECTIVE 6: Automated Insights & AI Recommendations =====================
+        st.exception(e)
+ # =================== OBJECTIVE 6: Automated Insights & AI Recommendations =====================
 st.subheader("OBJECTIVE 6: Instant Insights & Recommendations")
 
 # Membagi tampilan menjadi dua kolom