Enhance player eligibility checks and lineup processing in load_contest_file function

- Added debug prints to track position mapping and lineup processing, improving traceability during execution.
- Refined the logic for assigning players to required positions, ensuring accurate filling of both required and outfield positions.
- Implemented fallback handling for unassigned positions, enhancing data integrity and user experience.
- These changes contribute to ongoing efforts to improve player data handling within the application.

global_func/load_contest_file.py

@@ -61,49 +61,59 @@ def load_contest_file(upload, sport):
             # Create position mapping dictionary
             pos_dict = dict(zip(pos_df['Player'], pos_df['Pos']))
             
-            # Create empty columns for each position
-            if sport == 'MLB':
-                position_columns = ['SP1', 'SP2', 'C', '1B', '2B', '3B', 'SS', 'OF1', 'OF2', 'OF3']
-                for col in position_columns:
-                    df[col] = None
+            # Debug prints
+            print("\nPosition Dictionary:")
+            print(pos_dict)
+            
+            print("\nSample Lineup String:")
+            print(df['Lineup'].iloc[0])  # Print first lineup
+            
+            # Function to check if player is eligible for position
+            def is_eligible_for_position(player, target_pos):
+                if player not in pos_dict:
+                    print(f"Player not found in pos_dict: {player}")
+                    return False
+                player_positions = pos_dict[player].split('/')
+                print(f"Checking {player} for {target_pos}. Player positions: {player_positions}")
+                # Handle special cases
+                if target_pos.startswith('SP') and 'P' in player_positions:
+                    return True
+                if target_pos.startswith('OF') and 'OF' in player_positions:
+                    return True
+                return target_pos in player_positions
+            
+            # Process each lineup
+            for idx, row in df.iterrows():
+                # Get all players in the lineup
+                players = row['Lineup'].split(',')
+                players = [p.strip() for p in players if p.strip()]
+                print(f"\nProcessing lineup {idx}:")
+                print(f"Players found: {players}")
                 
-                # Function to check if player is eligible for position
-                def is_eligible_for_position(player, target_pos):
-                    if player not in pos_dict:
-                        return False
-                    player_positions = pos_dict[player].split('/')
-                    # Handle special cases
-                    if target_pos.startswith('SP') and 'P' in player_positions:
-                        return True
-                    if target_pos.startswith('OF') and 'OF' in player_positions:
-                        return True
-                    return target_pos in player_positions
+                # First pass: fill required positions
+                required_positions = ['SP1', 'SP2', 'C', '1B', '2B', '3B', 'SS']
+                for pos in required_positions:
+                    for player in players:
+                        if is_eligible_for_position(player, pos):
+                            print(f"Assigning {player} to {pos}")
+                            df.at[idx, pos] = player
+                            players.remove(player)
+                            break
+                    else:
+                        print(f"No player found for {pos}")
                 
-                # Process each lineup
-                for idx, row in df.iterrows():
-                    # Get all players in the lineup
-                    players = row['Lineup'].split(',')
-                    players = [p.strip() for p in players if p.strip()]
-                    
-                    # First pass: fill required positions (SP1, SP2, C, 1B, 2B, 3B, SS)
-                    required_positions = ['SP1', 'SP2', 'C', '1B', '2B', '3B', 'SS']
-                    for pos in required_positions:
-                        for player in players:
-                            if is_eligible_for_position(player, pos):
-                                df.at[idx, pos] = player
-                                players.remove(player)
-                                break
-                    
-                    # Second pass: fill OF positions with remaining players
-                    of_positions = ['OF1', 'OF2', 'OF3']
-                    for pos in of_positions:
-                        for player in players:
-                            if is_eligible_for_position(player, 'OF'):
-                                df.at[idx, pos] = player
-                                players.remove(player)
-                                break
-                        if not players:  # If no more players, fill with -1
-                            df.at[idx, pos] = '-1'
+                # Second pass: fill OF positions
+                of_positions = ['OF1', 'OF2', 'OF3']
+                for pos in of_positions:
+                    for player in players:
+                        if is_eligible_for_position(player, 'OF'):
+                            print(f"Assigning {player} to {pos}")
+                            df.at[idx, pos] = player
+                            players.remove(player)
+                            break
+                    else:
+                        print(f"No player found for {pos}, using -1")
+                        df.at[idx, pos] = '-1'
             
             cleaned_df = df.drop(columns=['EntryId', 'EntryName', 'TimeRemaining', 'Points', 'Lineup', 'Player', 'Pos', 'Own', 'FPTS', 'Salary', 'Team'])
             cleaned_df = cleaned_df[['BaseName', 'EntryCount', 'SP1', 'SP2', 'C', '1B', '2B', '3B', 'SS', 'OF1', 'OF2', 'OF3']]