Follow-ups from #35 #41

New Issue

aep · 2026-10-13T16:26:11+03:00

aep commented

2026-10-13 16:26:11 +03:00

Отложенные находки фичи #35 (PR #39), строки ИНН рядом с контрагентом. В скобках указано, откуда находка.

T2.4 — у _kind и _unrecognised_attributes есть побочные эффекты (ревью Task 2).
T3.5, T3.6 — докстринги _block_direction неточны (ревью Task 3).
T4.1 — имя владельца подсказки .ИНН не нормализуется по пробелам и NBSP (ревью Task 4).
T4.2 — дубль реквизита «Контрагенты.ИНН» в шапке не обрабатывается (ревью Task 4).
T4.4 — в classify.py нет комментария о порядке веток (ревью Task 4).
M-2 — договор, у которого вместо названия ИНН, в схеме «Контрагенты, Договоры» не защищён (финальное ревью).
M-9 (остаток) — стратегии иерархии indent и sums со строками ИНН не проверены тестами (финальное ревью).
N.1 — позиция реквизита берётся из _depths до склейки; на схеме с двумя смежными COUNTERPARTY возможна ошибка (пере-ревью fix-волны).
N.2 — единственный договор с детьми и теми же суммами без строки реквизита получает «ИНН не распознан» (пере-ревью; цена решения design #9/#22).
O-1 — флейк интеграционного test_roundtrip. Уже есть в #36, здесь для полноты.

Отложенные находки фичи #35 (PR #39), строки ИНН рядом с контрагентом. В скобках указано, откуда находка. - **T2.4** — у `_kind` и `_unrecognised_attributes` есть побочные эффекты (ревью Task 2). - **T3.5, T3.6** — докстринги `_block_direction` неточны (ревью Task 3). - **T4.1** — имя владельца подсказки `.ИНН` не нормализуется по пробелам и NBSP (ревью Task 4). - **T4.2** — дубль реквизита «Контрагенты.ИНН» в шапке не обрабатывается (ревью Task 4). - **T4.4** — в `classify.py` нет комментария о порядке веток (ревью Task 4). - **M-2** — договор, у которого вместо названия ИНН, в схеме «Контрагенты, Договоры» не защищён (финальное ревью). - **M-9 (остаток)** — стратегии иерархии `indent` и `sums` со строками ИНН не проверены тестами (финальное ревью). - **N.1** — позиция реквизита берётся из `_depths` до склейки; на схеме с двумя смежными COUNTERPARTY возможна ошибка (пере-ревью fix-волны). - **N.2** — единственный договор с детьми и теми же суммами без строки реквизита получает «ИНН не распознан» (пере-ревью; цена решения design #9/#22). - **O-1** — флейк интеграционного `test_roundtrip`. Уже есть в #36, здесь для полноты.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: RSHB/arm-tabular-parser#41