Update app.py

app.py

@@ -126,6 +126,12 @@ st.markdown("""
     [data-testid="stSidebar"] [data-testid="stExpander"] {
         color: var(--secondary-text-color) !important; 
     }
+    
+    /* Fix: Condense paragraph spacing in Quick Guide (Sidebar) */
+    .stSidebar .stExpanderContent p {
+        margin-block-start: 0.5em !important; 
+        margin-block-end: 0.5em !important;
+    }
 
     /* Scorecard Style (Tiles) */
     .scorecard-block {
@@ -183,7 +189,7 @@ skills_list = [
 
 skills_pattern = re.compile(r'\b(' + '|'.join(re.escape(skill) for skill in skills_list) + r')\b', re.IGNORECASE)
 
-# Helper functions for CV parsing
+# Helper functions for CV parsing (UNCHANGED)
 def extract_text_from_pdf(file):
     try:
         pdf_reader = PyPDF2.PdfReader(file)
@@ -279,7 +285,7 @@ def extract_skills(text):
 
 @st.cache_data
 def classify_and_summarize_batch(resume, job_description, _bert_tokenized, _t5_input, _t5_tokenized, _job_skills_set):
-    # Classification and Summary logic (unchanged)
+    # Classification and Summary logic (UNCHANGED CORE LOGIC)
     _, bert_model, t5_tokenizer, t5_model, device = st.session_state.models
     timeout = 60
     
@@ -343,13 +349,14 @@ def classify_and_summarize_batch(resume, job_description, _bert_tokenized, _t5_i
         elif suitability == "Irrelevant": color = "#F44336"
         else: color = "#FFC107"
 
-        return {"Suitability": suitability, "Data/Tech Related Skills Summary": final_summary, "Warning": warning or "None", "Suitability_Color": color}
+        # *** CHANGE 3: Renamed Suitability_Color to __style_color for clarity ***
+        return {"Suitability": suitability, "Data/Tech Related Skills Summary": final_summary, "Warning": warning or "None", "__style_color": color}
     except Exception as e:
-        return {"Suitability": "Error", "Data/Tech Related Skills Summary": "Failed to process profile", "Warning": str(e), "Suitability_Color": "#F44336"}
+        return {"Suitability": "Error", "Data/Tech Related Skills Summary": "Failed to process profile", "Warning": str(e), "__style_color": "#F44336"}
 
 @st.cache_data
 def generate_skill_pie_chart(resumes):
-    # Skill chart logic (unchanged)
+    # Skill chart logic (UNCHANGED)
     skill_counts = {}
     total_resumes = len([r for r in resumes if r.strip()])
     if total_resumes == 0: return None
@@ -372,7 +379,8 @@ def generate_skill_pie_chart(resumes):
     
     # Use dark theme settings for the chart
     plt.style.use('dark_background')
-    fig, ax = plt.subplots(figsize=(6, 4))
+    # *** CHANGE 1: Removed figsize to let Streamlit manage size and prevent flicker ***
+    fig, ax = plt.subplots() 
     colors = plt.cm.plasma(np.linspace(0.2, 0.9, len(labels))) 
     plt.rcParams['text.color'] = '#F8F8F8'
     wedges, texts, autotexts = ax.pie(sizes, labels=labels, autopct='%1.1f%%', startangle=90, colors=colors, textprops={'fontsize': 10, 'color': '#F8F8F8'})
@@ -645,27 +653,29 @@ def main():
             st.markdown("### 📋 Detailed Screening Results")
             
             # RANK 1: Color-Coded Table Rows using Pandas Styler
+            # *** CHANGE 3: The styling function now targets the internal __style_color column ***
             def style_suitability_row(row):
                 # Using light background color for dark theme
-                if row['Suitability_Color'] == '#4CAF50': # Relevant - Green
+                if row['__style_color'] == '#4CAF50': # Relevant - Green
                     return ['background-color: rgba(76, 175, 80, 0.15)'] * len(row)
-                elif row['Suitability_Color'] == '#F44336': # Irrelevant/Error - Red
+                elif row['__style_color'] == '#F44336': # Irrelevant/Error - Red
                     return ['background-color: rgba(244, 67, 54, 0.15)'] * len(row)
-                elif row['Suitability_Color'] == '#FFC107': # Uncertain - Yellow
+                elif row['__style_color'] == '#FFC107': # Uncertain - Yellow
                     return ['background-color: rgba(255, 193, 7, 0.15)'] * len(row)
                 else:
                     return [''] * len(row)
 
             # Apply styling and rename columns
-            display_df = results_df.rename(columns={'Data/Tech Related Skills Summary': 'PROFILE SUMMARY', 'Warning': 'FLAGGING REASON', 'Resume': 'PROFILE ID'})
+            # Note: display_df still contains '__style_color' for the styling function to use.
+            display_df = results_df.rename(columns={'Data/Tech Related Skills Summary': 'PROFILE SUMMARY', 'Warning': 'FLAGGING REASON', 'Resume': 'PROFILE ID', '__style_color': '__STYLE_COLOR_INTERNAL'})
             
             # Apply row styling (using the column that holds the hex color)
             styled_df = display_df.style.apply(style_suitability_row, axis=1)
             
-            # Remove the now-redundant color column for display
-            styled_df = styled_df.hide(subset=['Suitability_Color'], axis=1)
+            # Remove the now-internal-only color column from display
+            styled_df = styled_df.hide(subset=['__STYLE_COLOR_INTERNAL'], axis=1)
 
-            # Display the styled DataFrame. Note: column_config is not compatible with styled dataframes in Streamlit, but styler is compatible.
+            # Display the styled DataFrame.
             st.dataframe(
                 styled_df,
                 use_container_width=True
@@ -677,7 +687,8 @@ def main():
             
             with col_dl:
                 csv_buffer = io.StringIO()
-                results_df.drop(columns=['Suitability_Color']).to_csv(csv_buffer, index=False)
+                # *** CHANGE 3: Drop the internal column before download ***
+                results_df.drop(columns=['__style_color']).to_csv(csv_buffer, index=False)
                     
                 st.download_button(
                     "💾 Download Full Report (CSV)", 
@@ -692,7 +703,8 @@ def main():
                     if st.session_state.valid_resumes:
                         fig = generate_skill_pie_chart(st.session_state.valid_resumes)
                         if fig:
-                            st.pyplot(fig)
+                            # *** CHANGE 1: Added use_container_width for stability ***
+                            st.pyplot(fig, use_container_width=True) 
                             plt.close(fig)
                         else:
                             st.info("No recognized technical skills found in the profiles for charting.")