fix(tradein): sber series fallback continues on error + benchmark gated to ЕКБ region (#794 review)
This commit is contained in:
parent
3485b977f1
commit
468bdb052b
2 changed files with 30 additions and 27 deletions
|
|
@ -965,8 +965,8 @@ def _load_sber_index_series(db: Session, *, region: str) -> dict[date, float]:
|
|||
.all()
|
||||
)
|
||||
except Exception as exc:
|
||||
logger.warning("sber_price_index lookup failed (graceful, no-op): %s", exc)
|
||||
return {}
|
||||
logger.warning("sber_price_index lookup failed for %s (graceful): %s", dash, exc)
|
||||
continue
|
||||
if rows:
|
||||
return {r["period_month"]: float(r["index_value_rub_m2"]) for r in rows}
|
||||
return {}
|
||||
|
|
|
|||
|
|
@ -380,32 +380,35 @@ async def pull_sber_indices(
|
|||
latest_month.isoformat(),
|
||||
latest_val,
|
||||
)
|
||||
# Task B: best-effort reconciliation to our scrape-median asking ppm²
|
||||
try:
|
||||
our_median = db.execute(
|
||||
text("""
|
||||
SELECT percentile_cont(0.5)
|
||||
WITHIN GROUP (ORDER BY price_per_m2) AS m
|
||||
FROM listings
|
||||
WHERE is_active = true
|
||||
AND price_per_m2 BETWEEN 50000 AND 800000
|
||||
""")
|
||||
).scalar()
|
||||
if our_median is not None and latest_val:
|
||||
sber_latest = latest_val
|
||||
divergence_pct = (our_median - sber_latest) / sber_latest * 100
|
||||
logger.info(
|
||||
"sber_index benchmark [asking vs our median]: "
|
||||
"sber=%.0f our_median=%.0f divergence=%+.1f%% "
|
||||
"(data-quality signal)",
|
||||
sber_latest,
|
||||
our_median,
|
||||
divergence_pct,
|
||||
# our listings median is ЕКБ-only →
|
||||
# compare only against Свердловская обл. (REF_AREA 66)
|
||||
if ref_area == "66":
|
||||
# Task B: best-effort reconciliation to our scrape-median asking ppm²
|
||||
try:
|
||||
our_median = db.execute(
|
||||
text("""
|
||||
SELECT percentile_cont(0.5)
|
||||
WITHIN GROUP (ORDER BY price_per_m2) AS m
|
||||
FROM listings
|
||||
WHERE is_active = true
|
||||
AND price_per_m2 BETWEEN 50000 AND 800000
|
||||
""")
|
||||
).scalar()
|
||||
if our_median is not None and latest_val:
|
||||
sber_latest = latest_val
|
||||
divergence_pct = (our_median - sber_latest) / sber_latest * 100
|
||||
logger.info(
|
||||
"sber_index benchmark [asking vs our median]: "
|
||||
"sber=%.0f our_median=%.0f divergence=%+.1f%% "
|
||||
"(data-quality signal)",
|
||||
sber_latest,
|
||||
our_median,
|
||||
divergence_pct,
|
||||
)
|
||||
except Exception as exc:
|
||||
logger.debug(
|
||||
"sber benchmark reconciliation skipped (graceful): %s", exc
|
||||
)
|
||||
except Exception as exc:
|
||||
logger.debug(
|
||||
"sber benchmark reconciliation skipped (graceful): %s", exc
|
||||
)
|
||||
|
||||
except httpx.HTTPStatusError as exc:
|
||||
logger.error(
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue