changed the implementation of cv_short by adding form so the issue with min_exp is resolved

cv_short.py

@@ -85,137 +85,121 @@ class CVAnalyzer:
         return score_components
 
 
+def create_cv_shortlisting_page():
+    # logger.info("Starting CV shortlisting system")
 
-# def create_cv_shortlisting_page():
-#     logger.info("Starting CV shortlisting system")
+    # Initialize session state if not already initialized
+    # if 'jd_text' not in st.session_state:
+    #     st.session_state.jd_text = ""
+    # if 'min_years' not in st.session_state:
+    #     st.session_state.min_years = 3
+    # if 'required_skills_list' not in st.session_state:
+    #     st.session_state.required_skills_list = []
+    # if 'uploaded_files' not in st.session_state:
+    #     st.session_state.uploaded_files = None
+    # if 'results' not in st.session_state:
+    #     st.session_state.results = []
+    # if 'analysis_complete' not in st.session_state:
+    #     st.session_state.analysis_complete = False
 
-#     st.title("CV Shortlisting System")
-    
-#     # Reset analysis state when starting new analysis
-#     if 'analysis_started' not in st.session_state:
-#         st.session_state.analysis_started = False
+    # st.title("CV Shortlisting System")
     
-#     # Job Description Input
-#     st.header("Job Description")
-#     jd_text = st.text_area("Enter the job description",
-#                            value=st.session_state.jd_text if 'jd_text' in st.session_state else "")
+    # # Job Description Input
+    # st.header("Job Description")
+    # jd_text = st.text_area("Enter the job description", value=st.session_state.jd_text)
+    # if jd_text:
+    #     st.session_state.jd_text = jd_text
     
-#     if jd_text:
-#         st.session_state.jd_text = jd_text
+    # # Job Requirements Input
+    # st.header("Job Requirements")
+    # min_years = st.number_input("Minimum years of experience", 
+    #                            min_value=0, 
+    #                            value=st.session_state.min_years,
+    #                            )
     
-#     # Job Requirements Input
-#     st.header("Job Requirements")
-#     min_years = st.number_input("Minimum years of experience", 
-#                                min_value=0, 
-#                                value=st.session_state.min_years if 'min_years' in st.session_state else 3)
+    # required_skills = st.text_input("Required skills (comma-separated)", 
+    #                                value=','.join(st.session_state.required_skills_list) if st.session_state.required_skills_list else "")
     
-#     required_skills = st.text_input("Required skills (comma-separated)", 
-#                                    value=','.join(st.session_state.required_skills_list) if 'required_skills_list' in st.session_state else "")
+    # required_skills_list = [skill.strip() for skill in required_skills.split(",") if skill.strip()]
     
-#     required_skills_list = [skill.strip() for skill in required_skills.split(",") if skill.strip()]
-    
-#     # Update session state
-#     st.session_state.required_skills_list = required_skills_list
-#     st.session_state.min_years = min_years
+    # if required_skills_list:
+    #     st.session_state.required_skills_list = required_skills_list
+    # if min_years:
+    #     st.session_state.min_years = min_years
 
-#     # CV Upload
-#     st.header("Upload CVs")
-#     uploaded_files = st.file_uploader("Choose CV files", 
-#                                     accept_multiple_files=True, 
-#                                     type=['pdf', 'txt'], 
-#                                     key="cv_upload")
+    # # CV Upload
+    # st.header("Upload CVs")
+    # uploaded_files = st.file_uploader("Choose CV files", 
+    #                                 accept_multiple_files=True, 
+    #                                 type=['pdf', 'txt'], 
+    #                                 key="unique_cv_upload")
     
-#     if uploaded_files:
-#         st.session_state.uploaded_files = uploaded_files
-#         st.session_state.analysis_started = True
+    # if uploaded_files:
+    #     st.session_state.uploaded_files = uploaded_files
 
-#     # Analysis Button
-#     if st.button("Analyze CVs") and uploaded_files and jd_text:
-#         st.session_state.results = []  # Reset results
-#         st.session_state.processed_cvs = {}  # Reset processed CVs
-        
-        # with st.spinner('Analyzing CVs...'):
-        #     try:
-        #         analyzer = CVAnalyzer()
-                
-        #         # Prepare job requirements
-        #         job_requirements = {
-        #             "min_years_experience": st.session_state.min_years,
-        #             "required_skills": st.session_state.required_skills_list
-        #         }
+    # if st.button("Analyze CVs") and uploaded_files and jd_text:
+    #     # logger.info("Analyzing uploaded CVs")
+    #     with st.spinner('Analyzing CVs...'):
+    #         analyzer = CVAnalyzer()
+            
+    #         # Prepare job requirements
+    #         job_requirements = {
+    #             "min_years_experience": st.session_state.min_years,
+    #             "required_skills": st.session_state.required_skills_list
+    #         }
+            
+    #         results = []
+    #         st.session_state.results = []  # Reset results for new analysis
+
+    #         # Process each CV
+    #         for uploaded_file in uploaded_files:
+    #             cv_text = extr.process_file(uploaded_file)
                 
-        #         # Process each CV
-        #         for uploaded_file in uploaded_files:
-        #             cv_text = extr.process_file(uploaded_file)
+    #             try:
+    #                 candidates = analyzer.extract_cv_info(cv_text)
                     
-        #             try:
-        #                 # Extract CV information
-        #                 candidates = analyzer.extract_cv_info(cv_text)
+    #                 for candidate in candidates:
+    #                     match_scores = analyzer.calculate_match_score(
+    #                         candidate.__dict__, 
+    #                         job_requirements
+    #                     )
                         
-        #                 for idx, candidate in enumerate(candidates):
-        #                     # Calculate match scores
-        #                     match_scores = analyzer.calculate_match_score(
-        #                         candidate.__dict__, 
-        #                         job_requirements
-        #                     )
-                            
-        #                     # Store results
-        #                     result = {
-        #                         "Name": candidate.name or "Unknown",
-        #                         "Experience (Years)": candidate.years_of_exp or 0,
-        #                         "Skills": ", ".join(candidate.skills) if candidate.skills else "None",
-        #                         "Certifications": ", ".join(candidate.certifications) if candidate.certifications else "None",
-        #                         "Skills Match": f"{match_scores['skills_match']:.2%}",
-        #                         "Experience Match": f"{match_scores['experience_match']:.2%}",
-        #                         "Overall Score": f"{match_scores['overall_score']:.2%}"
-        #                     }
-                            
-        #                     st.session_state.results.append(result)
-                            
-        #                     # Store processed CV data for interview questions
-        #                     st.session_state.processed_cvs[f"{candidate.name}_{idx}"] = {
-        #                         "cv_text": cv_text,
-        #                         "candidate": candidate,
-        #                         "match_scores": match_scores
-        #                     }
-                            
-        #             except Exception as e:
-        #                 logger.error(f"Error processing CV: {str(e)}")
-        #                 st.error(f"Error processing CV: {str(e)}")
-                
-        #         # Mark analysis as complete
-        #         st.session_state.analysis_complete = True
-                
-        #         # Display results
-        #         if st.session_state.results:
-        #             df = pd.DataFrame(st.session_state.results)
-        #             df = df.sort_values("Overall Score", ascending=False)
-        #             st.dataframe(df)
-                    
-        #             # Save top candidates
-        #             st.session_state.top_candidates = df.head()
-        #         else:
-        #             logger.warning("No valid candidates found")
-        #             st.warning("No valid candidates found in the uploaded CVs")
-                    
-        #     except Exception as e:
-        #         logger.error(f"Analysis error: {str(e)}")
-        #         st.error(f"An error occurred during analysis: {str(e)}")
-        #         st.session_state.analysis_complete = False
-    
-#     # Display analysis status
-#     if st.session_state.get('analysis_complete', False):
-#         st.success("CV analysis complete! You can now proceed to generate interview questions.")
-
+    #                     result = {
+    #                         "Name": candidate.name or "Unknown",
+    #                         "Experience (Years)": candidate.years_of_exp or 0,
+    #                         "Skills": ", ".join(candidate.skills) if candidate.skills else "None",
+    #                         "Certifications": ", ".join(candidate.certifications) if candidate.certifications else "None",
+    #                         "Skills Match": f"{match_scores['skills_match']:.2%}",
+    #                         "Experience Match": f"{match_scores['experience_match']:.2%}",
+    #                         "Overall Score": f"{match_scores['overall_score']:.2%}"
+    #                     }
+                        
+    #                     results.append(result)
+    #                     st.session_state.results.append(result)
+                        
+    #             except Exception as e:
+    #                 st.error(f"Error processing CV: {str(e)}")
+    #                 # logger.error(f"Error processing CV: {str(e)}")
+            
+    #     # Display results
+    #     # logger.info(f"Displaying analyzed results for {len(results)} candidate(s)")
+        
+    #     if st.session_state.results:
+    #         df = pd.DataFrame(st.session_state.results)
+    #         df = df.sort_values("Overall Score", ascending=False)
+    #         st.dataframe(df)
+    #         st.session_state.analysis_complete = True
+    #     else:
+    #         # logger.warning("No valid candidates found in uploaded CVs")
+    #         st.error("No valid results found from CV analysis")
+    #         st.session_state.analysis_complete = False
 
-def create_cv_shortlisting_page():
-    # logger.info("Starting CV shortlisting system")
 
-    # Initialize session state if not already initialized
+# Initialize session state variables if they don't exist
     if 'jd_text' not in st.session_state:
-        st.session_state.jd_text = ""
+        st.session_state.jd_text = ''
     if 'min_years' not in st.session_state:
-        st.session_state.min_years = 3
+        st.session_state.min_years = 0
     if 'required_skills_list' not in st.session_state:
         st.session_state.required_skills_list = []
     if 'uploaded_files' not in st.session_state:
@@ -225,94 +209,81 @@ def create_cv_shortlisting_page():
     if 'analysis_complete' not in st.session_state:
         st.session_state.analysis_complete = False
 
-    st.title("CV Shortlisting System")
-    
-    # Job Description Input
-    st.header("Job Description")
-    jd_text = st.text_area("Enter the job description", value=st.session_state.jd_text)
-    if jd_text:
+    # Form for input
+    with st.form("job_description_form"):
+        # Job Description Input
+        st.header("Job Description")
+        jd_text = st.text_area("Enter the job description", value=st.session_state.jd_text)
+        
+        # Job Requirements Input
+        st.header("Job Requirements")
+        min_years = st.number_input("Minimum years of experience", min_value=0, value=st.session_state.min_years)
+        
+        required_skills = st.text_input("Required skills (comma-separated)", 
+                                        value=','.join(st.session_state.required_skills_list) if st.session_state.required_skills_list else "")
+        
+        # CV Upload
+        st.header("Upload CVs")
+        uploaded_files = st.file_uploader("Choose CV files", accept_multiple_files=True, type=['pdf', 'txt'], key="unique_cv_upload")
+        
+        # Submit Button
+        submit_button = st.form_submit_button(label="Analyze CVs")
+
+    # Update session state after form submission
+    if submit_button:
         st.session_state.jd_text = jd_text
-    
-    # Job Requirements Input
-    st.header("Job Requirements")
-    min_years = st.number_input("Minimum years of experience", 
-                               min_value=0, 
-                               value=st.session_state.min_years,
-                               )
-    
-    required_skills = st.text_input("Required skills (comma-separated)", 
-                                   value=','.join(st.session_state.required_skills_list) if st.session_state.required_skills_list else "")
-    
-    required_skills_list = [skill.strip() for skill in required_skills.split(",") if skill.strip()]
-    
-    if required_skills_list:
-        st.session_state.required_skills_list = required_skills_list
-    if min_years:
         st.session_state.min_years = min_years
-
-    # CV Upload
-    st.header("Upload CVs")
-    uploaded_files = st.file_uploader("Choose CV files", 
-                                    accept_multiple_files=True, 
-                                    type=['pdf', 'txt'], 
-                                    key="unique_cv_upload")
-    
-    if uploaded_files:
+        st.session_state.required_skills_list = [skill.strip() for skill in required_skills.split(",") if skill.strip()]
         st.session_state.uploaded_files = uploaded_files
 
-    if st.button("Analyze CVs") and uploaded_files and jd_text:
-        # logger.info("Analyzing uploaded CVs")
-        with st.spinner('Analyzing CVs...'):
-            analyzer = CVAnalyzer()
-            
-            # Prepare job requirements
-            job_requirements = {
-                "min_years_experience": st.session_state.min_years,
-                "required_skills": st.session_state.required_skills_list
-            }
-            
-            results = []
-            st.session_state.results = []  # Reset results for new analysis
-
-            # Process each CV
-            for uploaded_file in uploaded_files:
-                cv_text = extr.process_file(uploaded_file)
+        if st.session_state.uploaded_files and st.session_state.jd_text:
+            with st.spinner('Analyzing CVs...'):
+                analyzer = CVAnalyzer()
+                
+                # Prepare job requirements
+                job_requirements = {
+                    "min_years_experience": st.session_state.min_years,
+                    "required_skills": st.session_state.required_skills_list
+                }
                 
-                try:
-                    candidates = analyzer.extract_cv_info(cv_text)
+                results = []
+                st.session_state.results = []  # Reset results for new analysis
+
+                # Process each CV
+                for uploaded_file in st.session_state.uploaded_files:
+                    cv_text = extr.process_file(uploaded_file)
                     
-                    for candidate in candidates:
-                        match_scores = analyzer.calculate_match_score(
-                            candidate.__dict__, 
-                            job_requirements
-                        )
-                        
-                        result = {
-                            "Name": candidate.name or "Unknown",
-                            "Experience (Years)": candidate.years_of_exp or 0,
-                            "Skills": ", ".join(candidate.skills) if candidate.skills else "None",
-                            "Certifications": ", ".join(candidate.certifications) if candidate.certifications else "None",
-                            "Skills Match": f"{match_scores['skills_match']:.2%}",
-                            "Experience Match": f"{match_scores['experience_match']:.2%}",
-                            "Overall Score": f"{match_scores['overall_score']:.2%}"
-                        }
+                    try:
+                        candidates = analyzer.extract_cv_info(cv_text)
                         
-                        results.append(result)
-                        st.session_state.results.append(result)
-                        
-                except Exception as e:
-                    st.error(f"Error processing CV: {str(e)}")
-                    # logger.error(f"Error processing CV: {str(e)}")
-            
-        # Display results
-        # logger.info(f"Displaying analyzed results for {len(results)} candidate(s)")
-        
-        if st.session_state.results:
-            df = pd.DataFrame(st.session_state.results)
-            df = df.sort_values("Overall Score", ascending=False)
-            st.dataframe(df)
-            st.session_state.analysis_complete = True
-        else:
-            # logger.warning("No valid candidates found in uploaded CVs")
-            st.error("No valid results found from CV analysis")
-            st.session_state.analysis_complete = False
+                        for candidate in candidates:
+                            match_scores = analyzer.calculate_match_score(
+                                candidate.__dict__, 
+                                job_requirements
+                            )
+                            
+                            result = {
+                                "Name": candidate.name or "Unknown",
+                                "Experience (Years)": candidate.years_of_exp or 0,
+                                "Skills": ", ".join(candidate.skills) if candidate.skills else "None",
+                                "Certifications": ", ".join(candidate.certifications) if candidate.certifications else "None",
+                                "Skills Match": f"{match_scores['skills_match']:.2%}",
+                                "Experience Match": f"{match_scores['experience_match']:.2%}",
+                                "Overall Score": f"{match_scores['overall_score']:.2%}"
+                            }
+                            
+                            results.append(result)
+                            st.session_state.results.append(result)
+                            
+                    except Exception as e:
+                        st.error(f"Error processing CV: {str(e)}")
+                
+            # Display results
+            if st.session_state.results:
+                df = pd.DataFrame(st.session_state.results)
+                df = df.sort_values("Overall Score", ascending=False)
+                st.dataframe(df)
+                st.session_state.analysis_complete = True
+            else:
+                st.error("No valid results found from CV analysis")
+                st.session_state.analysis_complete = False