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

Follow-ups from #81 #84

New Issue
Open
opened 2026-10-28 20:46:28 +03:00 by aep · 0 comments
aep commented 2026-10-28 20:46:28 +03:00
Owner
Copy Link

Внутренние замечания финального ревью PR #82 (продуктовые — в Jira AURORA, эпик 1354):

  • M-4 — спека osv-parsing: «сторона без границы слова SHALL искаться только в остатке ячейки группы» строже кода — ^дт\s*вв\b ловит и отдельный подзаголовок «ДТВВ» под группой; поправить формулировку («в остатке ячейки группы или в подзаголовке под группой; в ячейках данных — нет»). Откуда: ревью #82, Minor-4.
  • M-5 — detect.classify_sheet (используется только в тестах) не знает о слабом продолжении формы (R-12): пометить в docstring как упрощённую модель или выразить через plan_sheets. Откуда: ревью #82, Minor-5.
  • M-6 — rules/osv_headers.yaml: комментарий к ^оборот(ы)?(?=\s*(дт|кт)), почему без \b (ради «ОборотыДтВВ»), в отличие от ^начало/^конец. Откуда: ревью #82, Minor-6.
Внутренние замечания финального ревью PR #82 (продуктовые — в Jira AURORA, эпик 1354): - M-4 — спека osv-parsing: «сторона без границы слова SHALL искаться только в остатке ячейки группы» строже кода — `^дт\s*вв\b` ловит и отдельный подзаголовок «ДТВВ» под группой; поправить формулировку («в остатке ячейки группы или в подзаголовке под группой; в ячейках данных — нет»). Откуда: ревью #82, Minor-4. - M-5 — `detect.classify_sheet` (используется только в тестах) не знает о слабом продолжении формы (R-12): пометить в docstring как упрощённую модель или выразить через `plan_sheets`. Откуда: ревью #82, Minor-5. - M-6 — `rules/osv_headers.yaml`: комментарий к `^оборот(ы)?(?=\s*(дт|кт))`, почему без `\b` (ради «ОборотыДтВВ»), в отличие от `^начало`/`^конец`. Откуда: ревью #82, Minor-6.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
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#84
No description provided.
Delete Branch "%!s()"

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?