RSHB/arm-tabular-parser
9
0
Fork 0
Code Issues 9 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Пустая детализация не проверяется соотношением подстрок (#45, AURORA-1527) #46

Merged
aep merged 1 commits from feat/45-subline-empty into main 2026-10-16 18:35:29 +03:00
Conversation 0 Commits 1 Files Changed 2 +40
aep commented 2026-10-16 18:27:04 +03:00
Owner
Copy Link

Closes #45. Связано: AURORA-1527, RSHB/arm-core#119, RSHB/arm-ka-fixtures#21 (golden), docs #115.

Что сделано

api/src/parser/statements/checks.py: мягкое соотношение «родитель = Σ пятизначных подстрок» по колонке проверяется, только если хотя бы одна подстрока родителя в ней имеет значение (amount не null; ноль — значение). Все пусты — ни нарушения, ни предупреждения. Остальные соотношения §7 и правило «пустая сумма = 0» не менялись. Словарь statement_checks.yaml не менялся (подстроки генерируются кодом). Поведение совпадает с arm-core.

Проверка

TDD (2 из 3 новых тестов были красными до правки). make check-full — зелёный (lint, unit, test-integration, docker-build); с golden из fixtures#21: make test-fixtures 1045 passed, make golden-check 148 без изменений. До merge fixtures#21 test-fixtures с main-снимками падает на 5 снимках — merge вместе.

Rulings I made

  • R-1: правило в коде, а не в yaml (подстроки генерируются кодом; флаг subline_check остаётся).
  • R-2: «значение» = amount is not None, как у arm-core; is_recognized/confidence не учитываются.

🤖 Generated with Claude Code

Closes #45. Связано: AURORA-1527, RSHB/arm-core#119, RSHB/arm-ka-fixtures#21 (golden), docs #115. ## Что сделано `api/src/parser/statements/checks.py`: мягкое соотношение «родитель = Σ пятизначных подстрок» по колонке проверяется, только если хотя бы одна подстрока родителя в ней имеет значение (`amount` не null; ноль — значение). Все пусты — ни нарушения, ни предупреждения. Остальные соотношения §7 и правило «пустая сумма = 0» не менялись. Словарь `statement_checks.yaml` не менялся (подстроки генерируются кодом). Поведение совпадает с arm-core. ## Проверка TDD (2 из 3 новых тестов были красными до правки). `make check-full` — зелёный (lint, unit, test-integration, docker-build); с golden из fixtures#21: `make test-fixtures` 1045 passed, `make golden-check` 148 без изменений. До merge fixtures#21 `test-fixtures` с main-снимками падает на 5 снимках — merge вместе. ## Rulings I made - R-1: правило в коде, а не в yaml (подстроки генерируются кодом; флаг `subline_check` остаётся). - R-2: «значение» = `amount is not None`, как у arm-core; `is_recognized`/`confidence` не учитываются. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
aep added 1 commit 2026-10-16 18:27:06 +03:00 
По колонке соотношение считается, только если хотя бы одна подстрока родителя имеет значение
(amount не null; ноль — значение). Совпадает с arm-core.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
aep merged commit 3f740c969b into main 2026-10-16 18:35:29 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
PotapovaA Raykov-MS SalminMP Veronika Yashin1994 aep grog maxim tsygankoviva
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: RSHB/arm-tabular-parser#46
No description provided.
Delete Branch "feat/45-subline-empty"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?