Enhance manual name matching process in find_name_mismatches function

- Streamlined the user interface for manual player matching by implementing a tabbed form layout, allowing users to select from the top 3 closest name matches.
- Improved the logic for processing both automatic and manual name matches, ensuring accurate updates to contest, ownership, and points data.
- Added success messages to confirm changes applied, enhancing user feedback during the matching process.

global_func/find_name_mismatches.py

@@ -42,71 +42,66 @@ def find_name_mismatches(contest_df, projections_df, ownership_df, fpts_df):
             st.success(f"Automatically matched '{player}' with '{match_name}' ({closest_matches[0][1]}% match)")
         else:
             players_to_process.append(player)
+
+    st.warning(f"Found {len(players_to_process)} players that need manual matching")
     
-    if players_to_process:
-        st.warning(f"Found {len(players_to_process)} players that need manual matching")
+    # Create a form for batch processing
+    with st.form("name_matching_form"):
+        # Create tabs for each player
+        tabs = st.tabs([f"Player {i+1}" for i in range(len(players_to_process))])
         
-        # Create a form for batch processing
-        with st.form("name_matching_form"):
-            # Create tabs for each player
-            tabs = st.tabs([f"Player {i+1}" for i in range(len(players_to_process))])
-            
-            # Dictionary to store selections
-            selections = {}
+        # Dictionary to store selections
+        selections = {}
+        
+        # Populate each tab
+        for i, player in enumerate(players_to_process):
+            with tabs[i]:
+                st.write(f"**Projection Name:** {player}")
+                
+                # Find the top 3 closest matches
+                closest_matches = process.extract(player, portfolio_players_list, limit=3)
+                
+                # Create radio buttons for selection
+                options = [f"{match[0]} ({match[1]}%)" for match in closest_matches]
+                options.append("None of these")
+                
+                selections[player] = st.radio(
+                    f"Select correct match:",
+                    options,
+                    key=f"radio_{player}"
+                )
+        
+        # Submit button for the entire form
+        submitted = st.form_submit_button("Apply All Changes")
+        
+        if submitted:
+            # Process automatic matches
+            for projection_name, contest_name in auto_matches.items():
+                for col in name_columns:
+                    contest_raw[col] = contest_raw[col].replace(contest_name, projection_name)
             
-            # Populate each tab
-            for i, player in enumerate(players_to_process):
-                with tabs[i]:
-                    st.write(f"**Projection Name:** {player}")
-                    
-                    # Find the top 3 closest matches
-                    closest_matches = process.extract(player, portfolio_players_list, limit=3)
-                    
-                    # Create radio buttons for selection
-                    options = [f"{match[0]} ({match[1]}%)" for match in closest_matches]
-                    options.append("None of these")
+            if contest_name in ownership_raw:
+                ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
+                
+            if contest_name in fpts_raw:
+                fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
                     
-                    selections[player] = st.radio(
-                        f"Select correct match:",
-                        options,
-                        key=f"radio_{player}"
-                    )
-            
-            # Submit button for the entire form
-            submitted = st.form_submit_button("Apply All Changes")
-            
-            if submitted:
-                # Process automatic matches
-                for projection_name, contest_name in auto_matches.items():
+            # Process manual selections
+            for projection_name, selection in selections.items():
+                if selection != "None of these":
+                    selected_name = selection.split(" (")[0]
                     for col in name_columns:
-                        contest_raw[col] = contest_raw[col].replace(contest_name, projection_name)
-                
-                if contest_name in ownership_raw:
-                    ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
+                        contest_raw[col] = contest_raw[col].replace(selected_name, projection_name)
                     
-                if contest_name in fpts_raw:
-                    fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
-                        
-                # Process manual selections
-                for projection_name, selection in selections.items():
-                    if selection != "None of these":
-                        selected_name = selection.split(" (")[0]
-                        for col in name_columns:
-                            contest_raw[col] = contest_raw[col].replace(selected_name, projection_name)
-                        
-                        if contest_name in ownership_raw:
-                            ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
-                        if contest_name in fpts_raw:
-                            fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
+                    if contest_name in ownership_raw:
+                        ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
+                    if contest_name in fpts_raw:
+                        fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
 
-                        st.success(f"Replaced '{selected_name}' with '{projection_name}'")
-                
-                ownership_dict = dict(zip(ownership_raw['Player'], ownership_raw['Own']))
-                fpts_dict = dict(zip(fpts_raw['Player'], fpts_raw['FPTS']))
-                
-                st.success("All changes applied successfully!")
+                    st.success(f"Replaced '{selected_name}' with '{projection_name}'")
             
-            # Return the current state if form hasn't been submitted yet
             ownership_dict = dict(zip(ownership_raw['Player'], ownership_raw['Own']))
             fpts_dict = dict(zip(fpts_raw['Player'], fpts_raw['FPTS']))
+            
+            st.success("All changes applied successfully!")
             return contest_raw, projections_raw, ownership_dict, fpts_dict