fix: [test] _extract_content_units 表格行计数包含非功能章节 - Closes #33 #35

2026-06-01T14:06:43+08:00

pzhang_zywl commented

2026-06-01 14:06:43 +08:00

Closes #33

问题

_extract_content_units() 统计 total_table_rows 时遍历了所有 sections，包括非功能章节（变更日志、术语解释、后方输入文件等）。

这些非功能章节的表格行（共 6 行）不可能也不应该被 function_units 覆盖，导致 table_coverage 分母虚高。

根因

functional_sections 正确过滤了，但 table_rows 统计没有。

影响

功能性表格行实际: 19 / 当前统计: 24 (多6行非功能) / 覆盖率: 13/24=54.2% 应为 13/19=68.4%

修复建议

仅统计 _is_functional_section 且 _has_section_content 的章节中的表格行。

阻塞: #21

[da-0601-0914]

[qe-agent: qa-01]

Closes #33 ## 问题 _extract_content_units() 统计 total_table_rows 时遍历了所有 sections，包括非功能章节（变更日志、术语解释、后方输入文件等）。这些非功能章节的表格行（共 6 行）不可能也不应该被 function_units 覆盖，导致 table_coverage 分母虚高。 ## 根因 functional_sections 正确过滤了，但 table_rows 统计没有。 ## 影响功能性表格行实际: 19 / 当前统计: 24 (多6行非功能) / 覆盖率: 13/24=54.2% 应为 13/19=68.4% ## 修复建议仅统计 _is_functional_section 且 _has_section_content 的章节中的表格行。阻塞: #21 [da-0601-0914] --- [qe-agent: qa-01]

pzhang_zywl added 2 commits 2026-06-01 14:06:45 +08:00

fix: table coverage only counts functional sections + specific missing row feedback - Closes #21

CI / test (pull_request) Successful in 8s

Details

93e13e947c

- _quick_validate: table rows only from functional sections
- Track specific missing rows with content for targeted feedback
- _build_coverage_feedback: includes missing row details
- Denominator: 24->18 rows, coverage: 54%->67%

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

test: _extract_content_units 仅统计功能章节表格行 - Closes #33

CI / test (pull_request) Successful in 9s

Details

119c08faca

非功能章节（变更日志、术语解释等）的表格行不可能被
function_units 覆盖，计入分母会导致覆盖率虚低。

修复: table_rows 统计仅在 _is_functional_section
且 _has_section_content 的章节中进行。

Table 覆盖率: 54.2% → 72.2% (24行→18行分母)

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

pzhang_zywl merged commit 4cf9f1d3e0 into main

2026-06-01 14:07:19 +08:00

pzhang_zywl referenced this issue from a commit

2026-06-01 14:07:21 +08:00

Merge pull request 'fix: [test] _extract_content_units 表格行计数包含非功能章节 - Closes #33' (#35) from test/issue-33 into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: zeekrAI/document_analyzer#35