Refactor name matching and export dictionary handling in app.py: streamline the name-to-ID mapping process by ensuring it only occurs when necessary, and enhance match handling by fixing the conversion to match_dict, improving overall efficiency and user experience during exports.

app.py

@@ -145,40 +145,42 @@ with tab1:
             
             # Update projections_df with any new matches
             st.session_state['projections_df'] = find_name_mismatches(st.session_state['portfolio'], st.session_state['projections_df'])
-            try:
-                name_id_map = dict(zip(
-                    st.session_state['csv_file']['Name'], 
-                    st.session_state['csv_file']['Name + ID']
-                ))
-            except:
-                name_id_map = dict(zip(
-                    st.session_state['csv_file']['Nickname'], 
-                    st.session_state['csv_file']['Id']
-                ))
-            
-            # Get all names at once
-            names = projections['player_names'].tolist()
-            choices = list(name_id_map.keys())
-            
-            # Process all names in one batch
-            matches = process.extract(
-                names,
-                choices,
-                scorer=fuzz.ratio,
-                score_cutoff=85,
-                limit=1  # Only get the best match
-            )
+            if 'export_dict' not in st.session_state and csv_file is not None:
+                try:
+                    name_id_map = dict(zip(
+                        st.session_state['csv_file']['Name'], 
+                        st.session_state['csv_file']['Name + ID']
+                    ))
+                except:
+                    name_id_map = dict(zip(
+                        st.session_state['csv_file']['Nickname'], 
+                        st.session_state['csv_file']['Id']
+                    ))
 
-            match_dict = {}
-            for name, match_list in zip(names, matches):
-                if match_list and match_list[0][1] >= 85:  # Check if we have a match and it meets threshold
-                    match_dict[name] = name_id_map[match_list[0][0]]  # Use the matched name to get the ID
-                else:
-                    match_dict[name] = name  # Keep original name if no good match
+                # Get all names at once
+                names = projections['player_names'].tolist()
+                choices = list(name_id_map.keys())
+
+                # Process all names in one batch
+                matches = process.extract(
+                    names,
+                    choices,
+                    scorer=fuzz.ratio,
+                    score_cutoff=85,
+                    limit=1  # Only get the best match
+                )
+
+                # Convert matches to dictionary - Fixed the match handling
+                match_dict = {}
+                for name, match_list in zip(names, matches):
+                    if match_list and match_list[0][1] >= 85:  # Check if we have a match and it meets threshold
+                        match_dict[name] = name_id_map[match_list[0][0]]  # Use the matched name to get the ID
+                    else:
+                        match_dict[name] = name  # Keep original name if no good match
 
-            # Apply the matches
-            projections['upload_match'] = projections['player_names'].map(match_dict)
-            st.session_state['export_dict'] = match_dict  # Use match_dict directly
+                # Apply the matches
+                projections['upload_match'] = projections['player_names'].map(match_dict)
+                st.session_state['export_dict'] = match_dict  # Use match_dict directly
             st.write(st.session_state['export_dict'])
                 
             st.session_state['origin_portfolio'] = st.session_state['portfolio'].copy()