Update app.py

app.py

@@ -18,10 +18,17 @@ def load_data_and_model():
         return None, None
     df = pd.read_csv(df_path)
     
+    # Standardize market value column name EARLY and DEDUPLICATE
+    for col in ['market_value_in_eur_x', 'market_value_in_eur_y', 'market_value_x', 'market_value_y']:
+        if col in df.columns:
+            df.rename(columns={col: 'market_value_in_eur'}, inplace=True)
+    
+    # CRITICAL: Drop any duplicate columns created by renaming to avoid "reindex on duplicate labels"
+    df = df.loc[:, ~df.columns.duplicated()].copy()
+    
     # Load model
     model = xgb.XGBRegressor()
     try:
-        # Try both common casing to be safe on Linux/HuggingFace
         if os.path.exists("fairvalue_xgboost.json"):
             model.load_model("fairvalue_xgboost.json")
         elif os.path.exists("FairValue_xgboost.json"):
@@ -43,11 +50,6 @@ if df is None:
     st.stop()
 
 # === Data Sanity Gate ===
-# Standardize market value column name for the model
-for col in ['market_value_in_eur_x', 'market_value_in_eur_y']:
-    if col in df.columns:
-        df.rename(columns={col: 'market_value_in_eur'}, inplace=True)
-
 if 'Age' in df.columns:
     df['Age'] = df['Age'].clip(15, 45)
 if 'Contract_Years_Left' in df.columns:
@@ -60,7 +62,7 @@ st.sidebar.header("Player Transfer Profile")
 
 input_mode = st.sidebar.radio("Input Mode", ["Select Existing Player", "Create Custom Player"])
 
-name_col = next((c for c in ['name', 'name_x', 'Player_Name'] if c in df.columns), None)
+name_col = next((c for c in ['name', 'name_x', 'Player_Name', 'Name'] if c in df.columns), None)
 
 if input_mode == "Select Existing Player":
     if name_col:
@@ -76,7 +78,10 @@ if input_mode == "Select Existing Player":
     
     contract_default = max(0.5, min(6.0, float(player_data['Contract_Years_Left'].iloc[0])))
     age_default = max(16, min(40, int(player_data['Age'].iloc[0])))
-    injury_default = max(0, min(500, int(player_data.get('Injury_Days_Total_24m', [10])[0])))
+    injury_col = next((c for c in ['Injury_Days_Total_24m', 'Injury_Days'] if c in player_data.columns), None)
+    injury_default = 0
+    if injury_col:
+        injury_default = max(0, min(500, int(player_data[injury_col].iloc[0])))
     
     contract_years = st.sidebar.slider("Contract Years Remaining", 0.5, 6.0, contract_default, 0.5)
     age = st.sidebar.slider("Age", 16, 40, age_default)
@@ -100,7 +105,8 @@ asking_price = st.sidebar.number_input("Selling Club Asking Price (£m)", 1.0, 2
 # Apply overrides
 player_data['Contract_Years_Left'] = contract_years
 player_data['Age'] = age
-player_data['Injury_Days_Total_24m'] = injuries if 'Injury_Days_Total_24m' in player_data.columns else injuries
+if 'Injury_Days_Total_24m' in player_data.columns:
+    player_data['Injury_Days_Total_24m'] = injuries
 
 # Prepare inference payload
 expected_cols = model.feature_names_in_