Update app.py

app.py

@@ -15,8 +15,7 @@ import streamlit as st
 from src.csv_enrichment import (
     TARGET_COLUMNS,
     EnrichmentConfig,
-    enrich_csv,                        # use canonical name (alias also works)
-    lookup_fund_metric_value,
+    enrich_csv,
 )
 from src.data_engine import run_data_engine
 
@@ -222,7 +221,7 @@ def main() -> None:
             <p class="mf-helper">
               Upload your standard fund universe CSV
               (<code>Fund</code>, <code>Benchmark Type</code>, CAGR columns, etc.).<br>
-              <strong>Firecrawl/Tavily is used only for missing P/E and P/B</strong> —
+              <strong>P/E and P/B are computed from AMFI monthly holdings (active funds) or NSE index API (index funds)</strong> —
               all risk metrics (Alpha, Sharpe, Sortino, etc.) are computed directly from NAV history.
             </p>
             """,
@@ -245,8 +244,8 @@ def main() -> None:
             )
 
         st.info(
-            "Pipeline: **Scheme code resolution → NAV engine (parallel, 12 workers) "
-            "→ PE/PB web lookup → category median fallback → scoring engine**"
+            "Pipeline: **Scheme code resolution → NAV engine (parallel) "
+            "→ PE/PB via AMFI holdings + NSE API → category median fallback → scoring engine**"
         )
 
         st.markdown(
@@ -334,10 +333,8 @@ def main() -> None:
                             config=EnrichmentConfig(
                                 enabled=True,
                                 max_cells=None,
-                                min_confidence=0.65,
-                                resolve_scheme_codes=True,   # ← parallel scheme resolution
-                                enable_nav_engine=True,      # ← parallel NAV engine (12 workers)
-                                web_search_pe_pb_only=True,  # ← only PE/PB uses API credits
+                                resolve_scheme_codes=True,
+                                enable_nav_engine=True,
                                 impute_unresolved=True,
                             ),
                             live_callback=_live_sink,
@@ -351,7 +348,6 @@ def main() -> None:
                             f"enrichment_done "
                             f"checked={enrichment.examined_cells} "
                             f"nav={enrichment.nav_cells} "
-                            f"web={enrichment.web_cells} "
                             f"imputed={enrichment.imputed_cells} "
                             f"skipped={enrichment.skipped_cells} "
                             f"codes={enrichment.resolved_codes} "
@@ -362,7 +358,6 @@ def main() -> None:
                             f"   ✅ Enrichment done in **{enrich_secs:.0f}s** — "
                             f"checked {enrichment.examined_cells} cells, "
                             f"NAV filled {enrichment.nav_cells}, "
-                            f"web filled {enrichment.web_cells}, "
                             f"imputed {enrichment.imputed_cells}"
                         )
 
@@ -418,7 +413,6 @@ def main() -> None:
                 enrichment_summary = (
                     f"Enrichment: {enrichment.examined_cells} cells checked — "
                     f"NAV filled {enrichment.nav_cells}, "
-                    f"web filled {enrichment.web_cells}, "
                     f"imputed {enrichment.imputed_cells}, "
                     f"skipped {enrichment.skipped_cells}."
                 )
@@ -491,8 +485,8 @@ def main() -> None:
             |---|---|
             | **0 — Scheme resolution** | Parallel fuzzy-match of missing AMFI scheme codes (8 threads) |
             | **1 — NAV engine** | Trailing 3Y risk metrics computed from mfapi NAV history (12 threads) |
-            | **2 — PE/PB web search** | Tavily (primary) or Firecrawl (fallback) — only for missing P/E and P/B |
-            | **3 — Median impute** | Category median fills remaining gaps for young/NA funds |
+            | **2 — PE/PB engine** | Active funds: AMFI monthly holdings weighted PE/PB (same as Groww). Index funds: NSE index API |
+            | **3 — Median impute** | Category median fills remaining gaps for ≥3Y funds. Young funds (<3Y) marked NA |
             | **4 — Scoring** | Top/Bottom 10 per category, 10-point weighted model |
             | **5 — Excel export** | Conditional formatting, quartile bands, benchmark rows |