fix(tradein): correct SberIndex hedonic/asking dataset filter dims + 404 warning (#902) #1724

Merged

lekss361 merged 1 commit from fix/sber-dataset-paths-902 into main

2026-06-17 19:58:49 +00:00

lekss361 commented

2026-06-17 19:51:59 +00:00

Owner

Проблема (#902)

Hedonic + asking слаги 404 на /api/sowa, тянулся только real_estate_deals.

Root cause (Playwright route-capture на живых дашбордах)

Путь /dataset/v1/<slug> был верный — баг в filter-измерениях (#794): hedonic слал REAL_ESTATE_NOVELTY=3 — невалидный код → 404. Перехват live-200 показал валидные: REAL_ESTATE_NOVELTY=1, REAL_ESTATE_TYPE ∈ {1=новостройка, 2=вторичка}, SOURCE=SI. Asking (SOURCE=DK, PRICE_TYPE=1) уже совпадал с 200-серией.

Решение

Hedonic → SOURCE=SI, REAL_ESTATE_TYPE=2 (вторичка), REAL_ESTATE_NOVELTY=1. Выбрана вторичка: estimator работает по вторичке (listing_segment='vtorichka'), а PK (city, period_month, dashboard) = одна серия на слаг (две слились бы на upsert). segment_field hedonic: real_estate_novelty→real_estate_type.
404 в pull_sber_indices → logger.warning со слагом+фильтром+хинтом «dataset-path invalid? re-capture», вместо тихого error-counter.

Verify

ruff clean; pytest -k sber → 32 passed (+тест build_sber_route byte-for-byte против 4 captured base64-роутов, +404-warning тест).

⚠️ Глазами перед мержем

PRICE_TYPE=1 для asking — семантика новостройка/вторичка не задокументирована (оба возвращают 200). Если 1=новостройка, asking-бенчмарк будет не вторичка-aligned. Уточнить при активации.

Closes #902

## Проблема (#902) Hedonic + asking слаги 404 на `/api/sowa`, тянулся только `real_estate_deals`. ## Root cause (Playwright route-capture на живых дашбордах) Путь `/dataset/v1/<slug>` был верный — баг в **filter-измерениях** (#794): hedonic слал `REAL_ESTATE_NOVELTY=3` — невалидный код → 404. Перехват live-200 показал валидные: `REAL_ESTATE_NOVELTY=1`, `REAL_ESTATE_TYPE ∈ {1=новостройка, 2=вторичка}`, `SOURCE=SI`. Asking (`SOURCE=DK, PRICE_TYPE=1`) уже совпадал с 200-серией. ## Решение - Hedonic → `SOURCE=SI, REAL_ESTATE_TYPE=2 (вторичка), REAL_ESTATE_NOVELTY=1`. Выбрана вторичка: estimator работает по вторичке (`listing_segment='vtorichka'`), а PK `(city, period_month, dashboard)` = одна серия на слаг (две слились бы на upsert). `segment_field` hedonic: `real_estate_novelty`→`real_estate_type`. - 404 в `pull_sber_indices` → `logger.warning` со слагом+фильтром+хинтом «dataset-path invalid? re-capture», вместо тихого error-counter. ## Verify `ruff` clean; `pytest -k sber` → **32 passed** (+тест `build_sber_route` byte-for-byte против 4 captured base64-роутов, +404-warning тест). ## ⚠️ Глазами перед мержем `PRICE_TYPE=1` для asking — семантика новостройка/вторичка не задокументирована (оба возвращают 200). Если 1=новостройка, asking-бенчмарк будет не вторичка-aligned. Уточнить при активации. Closes #902

lekss361 added 1 commit 2026-06-17 19:52:00 +00:00