[tradein][ux] confidence_explanation противоречит счётчикам (шаблонный текст) #695

2026-05-30 10:22:39 +00:00

label 2026-05-30 10:22:13 +00:00

lekss361 commented

Owner

auto-analyst triage: single-scope backend, clear acceptance (explanation text must match actual n/tier/addresses), no deps → status/ready.

bot-backend self-assigned this 2026-05-30 10:39:48 +00:00

bot-backend added

status/wip

and removed

labels 2026-05-30 10:39:49 +00:00

bot-backend was unassigned by bot-qa

2026-05-30 15:00:22 +00:00

bot-qa added the

2026-05-30 15:00:22 +00:00

label 2026-05-30 15:00:22 +00:00

bot-qa commented

Collaborator

⚠️ Stale claim released: issue stuck in status/wip for 4h (previously assigned to bot-backend). Label transition: status/wip → status/ready. Worker likely crashed.

bot-backend self-assigned this 2026-05-30 15:03:56 +00:00

bot-backend removed the

2026-05-30 15:07:57 +00:00

label 2026-05-30 15:03:58 +00:00

bot-backend referenced this issue from a commit

fix(tradein): confidence_explanation no longer contradicts counters (#695)

bot-backend referenced this issue

2026-05-30 15:08:07 +00:00

fix(tradein): confidence_explanation no longer contradicts counters (#695) #738

bot-backend added

2026-05-30 15:08:18 +00:00

and removed

status/wip

labels 2026-05-30 15:08:14 +00:00

bot-backend commented

Author

Collaborator

Working on this in PR #738 (re-picked after the stale-claim release — no prior PR existed). Root cause: when the same-building/micro-radius anchor builds the headline, the explanation concatenated the radius base text («Найдено N из M разных адресов», count=n_analogs) + analog_tier note + the anchor sentence («по anchor['n'] из того же дома») — different populations → the «4 vs 5» / «разных адресов vs того же дома» contradiction. Fix: anchor sentence becomes authoritative (replace, not append; repair_note preserved). 52 tests (+2) + 228 regression green, ruff clean. Radius path unchanged.

Working on this in PR #738 (re-picked after the stale-claim release — no prior PR existed). Root cause: when the same-building/micro-radius anchor builds the headline, the explanation concatenated the radius base text («Найдено N из M разных адресов», count=`n_analogs`) + `analog_tier` note + the anchor sentence («по `anchor['n']` из того же дома») — different populations → the «4 vs 5» / «разных адресов vs того же дома» contradiction. Fix: anchor sentence becomes authoritative (replace, not append; `repair_note` preserved). 52 tests (+2) + 228 regression green, ruff clean. Radius path unchanged.

bot-reviewer referenced this issue from a commit

2026-05-30 15:10:32 +00:00

fix(tradein): confidence_explanation no longer contradicts counters (#695) (#738)

bot-reviewer added

and removed

2026-05-30 15:17:33 +00:00

labels 2026-05-30 15:10:38 +00:00

bot-qa commented

Collaborator

❌ QA smoke FAIL — explanation всё ещё противоречит счётчикам (PR #738 пофиксил только дубль ВНУТРИ строки, не cross-field). Acceptance не выполнен.

Repro — Олимп penthouse (207М, premium showcase для демо), result-страница ?id=…&brand=praktika:
На ОДНОЙ странице одновременно:

из n_analogs=4: «Аналогов 4», «Аналогов в выборке 4», «4 аналог.», «Объявлений по аналогам 4 шт из 4 источников», и явный артефакт «Показано 5 из 4 объявлений»
из anchor-счёта 5: «Оценка построена по 5 аналогам из того же дома» (×2), «По 5 аналогам» (гистограмма/распределение)
фактическая таблица аналогов = 5 строк (все «Олимпийская набережная, 13»).

→ пользователь видит «Аналогов 4» рядом с «по 5 аналогам» и «Показано 5 из 4». Это ровно исходная жалоба #695 («explanation противоречит счётчикам»), и она НЕ устранена.

Что PR #738 сделал: убрал из строки explanation конфликтующий клоз «Найдено 4 из 4 разных адресов», оставив только «по 5 … того же дома». Строка перестала быть self-contradictory ✓ — но поле n_analogs осталось 4 (radius listings_clean), и все UI-счётчики из него (включая «Показано N из M») по-прежнему = 4, тогда как реально показано/построено 5.

Root cause: same-building anchor (#691/#694) подменяет headline и список аналогов на 5 комплов дома, но n_analogs считается по радиусному пути (=4) и не синхронизируется с anchor-популяцией. Малышева прошла только потому, что там radius-count совпал с anchor-count (14=14) — мимикрия, не фикс.

Fix: когда anchor_tier is not None, выставлять n_analogs = числу anchor-комплов (= длине показываемого analogs), чтобы все счётчики («Аналогов», «Показано N из M», «по N аналогам», гистограмма) согласовались на 5. (Контракт n_analogs из #698 — «полное число найденных» — для anchor-пути это и есть anchor-комплы.)

Скрин: job-tmp qa695-contradiction.jpeg. Малышева (14=14) — чисто; фейлит anchor-кейс с radius≠anchor (Олимп).

→ status/qa → status/needs-fix, assignee @bot-backend (автор PR #738), fixup.

❌ **QA smoke FAIL** — explanation всё ещё противоречит счётчикам (PR #738 пофиксил только дубль ВНУТРИ строки, не cross-field). Acceptance не выполнен. **Repro — Олимп penthouse (207М, premium showcase для демо), result-страница `?id=…&brand=praktika`:** На ОДНОЙ странице одновременно: - из `n_analogs=4`: «Аналогов **4**», «Аналогов в выборке **4**», «**4** аналог.», «Объявлений по аналогам **4** шт из 4 источников», и явный артефакт **«Показано 5 из 4 объявлений»** - из anchor-счёта 5: «Оценка построена по **5** аналогам из того же дома» (×2), «По **5** аналогам» (гистограмма/распределение) - фактическая таблица аналогов = **5** строк (все «Олимпийская набережная, 13»). → пользователь видит «Аналогов 4» рядом с «по 5 аналогам» и «Показано 5 из 4». Это ровно исходная жалоба #695 («explanation противоречит счётчикам»), и она НЕ устранена. **Что PR #738 сделал:** убрал из строки explanation конфликтующий клоз «Найдено 4 из 4 разных адресов», оставив только «по 5 … того же дома». Строка перестала быть self-contradictory ✓ — но **поле `n_analogs` осталось 4** (radius `listings_clean`), и все UI-счётчики из него (включая «Показано N из M») по-прежнему = 4, тогда как реально показано/построено 5. **Root cause:** same-building anchor (#691/#694) подменяет headline и список аналогов на 5 комплов дома, но `n_analogs` считается по радиусному пути (=4) и не синхронизируется с anchor-популяцией. Малышева прошла только потому, что там radius-count совпал с anchor-count (14=14) — мимикрия, не фикс. **Fix:** когда `anchor_tier is not None`, выставлять `n_analogs` = числу anchor-комплов (= длине показываемого `analogs`), чтобы все счётчики («Аналогов», «Показано N из M», «по N аналогам», гистограмма) согласовались на 5. (Контракт `n_analogs` из #698 — «полное число найденных» — для anchor-пути это и есть anchor-комплы.) Скрин: job-tmp `qa695-contradiction.jpeg`. Малышева (14=14) — чисто; фейлит anchor-кейс с radius≠anchor (Олимп). → `status/qa` → `status/needs-fix`, assignee @bot-backend (автор PR #738), fixup.

bot-qa added

status/needs-fix

and removed

2026-05-30 15:23:41 +00:00

labels 2026-05-30 15:17:39 +00:00

bot-backend referenced this issue from a commit

fix(tradein): sync n_analogs to anchor count when anchor drives headline (#695)

bot-backend referenced this issue

2026-05-30 15:23:54 +00:00

fix(tradein): sync n_analogs to anchor count when anchor drives headline (#695) #739

bot-backend added

2026-05-30 15:24:07 +00:00

and removed

status/needs-fix

labels 2026-05-30 15:24:03 +00:00

bot-backend commented

Author

Collaborator

fixup (QA-fail on #738) → new PR #739. Spot-on diagnosis — thank you. #738 fixed the explanation string but n_analogs stayed = radius count while the anchor replaced the headline/list with the house comps, so all UI counters (incl. «Показано 5 из 4») still read the radius number; Малышева passed only because radius==anchor (mimicry). Fix: anchor_tier is not None → n_analogs = anchor['n']. New test reproduces the exact failure shape (radius=5 ≠ anchor=3 → n_analogs==3, «по 3 аналогам» agree). 54 tests (+2) + 234 regression green. Re-QA the Олимп penthouse after merge — «Аналогов»/«Показано N из M»/«по N»/histogram should all read 5.

fixup (QA-fail on #738) → new PR #739. Spot-on diagnosis — thank you. #738 fixed the explanation *string* but `n_analogs` stayed = radius count while the anchor replaced the headline/list with the house comps, so all UI counters (incl. «Показано 5 из 4») still read the radius number; Малышева passed only because radius==anchor (mimicry). Fix: `anchor_tier is not None` → `n_analogs = anchor['n']`. New test reproduces the exact failure shape (radius=5 ≠ anchor=3 → n_analogs==3, «по 3 аналогам» agree). 54 tests (+2) + 234 regression green. Re-QA the Олимп penthouse after merge — «Аналогов»/«Показано N из M»/«по N»/histogram should all read 5.

bot-reviewer referenced this issue from a commit

2026-05-30 15:25:37 +00:00

fix(tradein): sync n_analogs to anchor count when anchor drives headline (#695) (#739)

bot-reviewer added

and removed

2026-05-30 15:29:04 +00:00

labels 2026-05-30 15:25:47 +00:00

bot-qa commented

Collaborator

✅ QA re-smoke PASS после fixup (PR #739, deployed @ 34a13d7). Контрадикция устранена end-to-end.

API (POST /estimate Олимп penthouse): n_analogs=5, shown analogs=5, explanation «по 5 аналогам из того же дома» — n_analogs == len(analogs) ✓

Frontend (result-страница Олимп, premium showcase) — все счётчики согласованы на 5:

было (FAIL)	стало (PASS)
«Показано 5 из 4 объявлений»	«Показано 5 из 5 объявлений» ✓
«Аналогов 4»	«Аналогов 5» ✓
«по 5 / Аналогов 4» рассинхрон	«по 5 аналогам» + «построена по 5 аналогам» + «По 5 аналогам» (гистограмма) ✓

has4-artifact (regex на «Показано 5 из 4» / «Аналогов 4») → false.

Fix верный: anchor_tier is not None → n_analogs = anchor['n'] — теперь все UI-счётчики читают anchor-популяцию (= показываемые комплы), а не radius-count. Малышева (radius==anchor) остаётся чистой. 54 unit (+2, incl. radius≠anchor failure-shape) + 234 regression зелёные. → status/done, closing.

✅ **QA re-smoke PASS** после fixup (PR #739, deployed @ `34a13d7`). Контрадикция устранена end-to-end. **API** (`POST /estimate` Олимп penthouse): `n_analogs=5`, shown analogs=5, explanation «по 5 аналогам из того же дома» — `n_analogs == len(analogs)` ✓ **Frontend** (result-страница Олимп, premium showcase) — все счётчики согласованы на 5: | было (FAIL) | стало (PASS) | |---|---| | «Показано **5 из 4** объявлений» | «Показано **5 из 5** объявлений» ✓ | | «Аналогов **4**» | «Аналогов **5**» ✓ | | «по 5 / Аналогов 4» рассинхрон | «по 5 аналогам» + «построена по 5 аналогам» + «По 5 аналогам» (гистограмма) ✓ | `has4-artifact` (regex на «Показано 5 из 4» / «Аналогов 4») → **false**. Fix верный: `anchor_tier is not None → n_analogs = anchor['n']` — теперь все UI-счётчики читают anchor-популяцию (= показываемые комплы), а не radius-count. Малышева (radius==anchor) остаётся чистой. 54 unit (+2, incl. radius≠anchor failure-shape) + 234 regression зелёные. → `status/done`, closing.

bot-qa added

status/done

and removed