Update app.py

app.py

@@ -481,36 +481,44 @@ def search_tickers_cb(q: str):
     opts = [f"{h['symbol']}  |  {h['name']}  |  {h['exchange']}" for h in hits]
     return "Select a symbol and click Add", opts
 
+# PATCH 1: lenient add (no validation/network)
 def add_symbol(selection: str, table: pd.DataFrame):
     if not selection:
         return table, "Pick a row from Matches first"
     symbol = selection.split("|")[0].strip().upper()
-    current = [] if table is None or len(table) == 0 else [str(x).upper() for x in table["ticker"].tolist() if str(x) != "nan"]
+
+    current = [] if table is None or len(table) == 0 else [
+        str(x).upper() for x in table["ticker"].tolist() if str(x) != "nan"
+    ]
+    # de-dup and append without validation here
     tickers = current if symbol in current else current + [symbol]
-    val = validate_tickers(tickers, years=DEFAULT_LOOKBACK_YEARS)
-    tickers = [t for t in tickers if t in val]
+
     amt_map = {}
     if table is not None and len(table) > 0:
         for _, r in table.iterrows():
             t = str(r.get("ticker", "")).upper()
             if t in tickers:
                 amt_map[t] = float(pd.to_numeric(r.get("amount_usd", 0.0), errors="coerce") or 0.0)
-    new_table = pd.DataFrame({"ticker": tickers, "amount_usd": [amt_map.get(t, 0.0) for t in tickers]})
-    msg = f"Added {symbol}" if symbol in tickers else f"{symbol} not valid"
-    if len(new_table) > MAX_TICKERS:
-        new_table = new_table.iloc[:MAX_TICKERS]
+
+    tickers = tickers[:MAX_TICKERS]
+    new_table = pd.DataFrame({
+        "ticker": tickers,
+        "amount_usd": [amt_map.get(t, 0.0) for t in tickers]
+    })
+
+    msg = f"Added {symbol}" if symbol in new_table["ticker"].tolist() else f"{symbol} not added"
+    if len(current) + 1 > MAX_TICKERS and symbol not in current:
         msg = f"Reached max of {MAX_TICKERS}"
     return new_table, msg
 
+# PATCH 2: keep user entries; no validation here
 def lock_ticker_column(tb: pd.DataFrame):
     if tb is None or len(tb) == 0:
         return pd.DataFrame(columns=["ticker", "amount_usd"])
-    tickers = [str(x).upper() for x in tb["ticker"].tolist()]
+    tickers = [str(x).upper().strip() for x in tb["ticker"].tolist()]
     amounts = pd.to_numeric(tb["amount_usd"], errors="coerce").fillna(0.0).tolist()
-    val = validate_tickers(tickers, years=DEFAULT_LOOKBACK_YEARS)
-    tickers = [t for t in tickers if t in val]
     amounts = amounts[:len(tickers)] + [0.0] * max(0, len(tickers) - len(amounts))
-    return pd.DataFrame({"ticker": tickers, "amount_usd": amounts})
+    return pd.DataFrame({"ticker": tickers[:MAX_TICKERS], "amount_usd": amounts[:MAX_TICKERS]})
 
 def set_horizon(years: float):
     y = max(1.0, min(100.0, float(years)))
@@ -533,9 +541,10 @@ def compute(years_lookback: int, table: pd.DataFrame,
     if len(symbols) == 0:
         return None, "Add at least one ticker", "Universe empty", empty_positions_df(), empty_suggest_df(), None
 
+    # Validation happens HERE (may hit network). If none validate, we'll message the user.
     symbols = validate_tickers(symbols, years_lookback)
     if len(symbols) == 0:
-        return None, "Could not validate any tickers", "Universe invalid", empty_positions_df(), empty_suggest_df(), None
+        return None, "Could not validate any tickers (check symbols or network)", "Universe invalid", empty_positions_df(), empty_suggest_df(), None
 
     global UNIVERSE
     UNIVERSE = list(sorted(set([s for s in symbols if s != MARKET_TICKER] + [MARKET_TICKER])))[:MAX_TICKERS]