Refine comments and logic in load_contest_file function for lineup processing

- Updated comments to clarify the distinction between filling required positions and outfield positions, enhancing code readability.
- Adjusted the logic for assigning players to outfield positions to ensure it checks for 'OF' in the position dictionary, improving accuracy in player assignments.
- These changes contribute to ongoing efforts to enhance code clarity and data handling within the application.

global_func/load_contest_file.py

@@ -90,7 +90,7 @@ def load_contest_file(upload, sport):
                 print(f"\nProcessing lineup {idx}:")
                 print(f"Players found: {players}")
                 
-                # First pass: fill required positions
+                # First pass: fill required positions (excluding OF)
                 required_positions = ['SP1', 'SP2', 'C', '1B', '2B', '3B', 'SS']
                 for pos in required_positions:
                     for player in players:
@@ -102,11 +102,11 @@ def load_contest_file(upload, sport):
                     else:
                         print(f"No player found for {pos}")
                 
-                # Second pass: fill OF positions
+                # 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'):
+                        if 'OF' in pos_dict.get(player, '').split('/'):
                             print(f"Assigning {player} to {pos}")
                             df.at[idx, pos] = player
                             players.remove(player)