zeekrAI/document_analyzer
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

fix: 表格行覆盖率仅统计功能章节 + 具体缺失行反馈 - Closes #21 #34

Closed
pzhang_zywl wants to merge 0 commits from dev/issue-21-table-coverage-feedback into main
pull from: dev/issue-21-table-coverage-feedback
merge into: zeekrAI:main
Conversation 1 Commits - Files Changed -
pzhang_zywl commented 2026-06-01 14:04:28 +08:00
Owner
Copy Link
Copy Source

Summary

  • _quick_validate: 表格行总数仅从功能章节统计（_is_functional_section + _has_section_content）
  • 追踪具体缺失表格行的 (section, row, key, value) 信息
  • _build_coverage_feedback: 反馈包含缺失表格行的具体内容

Impact

指标 修复前 修复后
表格行分母 24 (含6行非功能) 18 (仅功能章节)
表格行覆盖率 54.2% 67%

Test

  • pytest 全量通过 (102 passed, 13 skipped)

Closes #21

[da-0601-0914]

## Summary - _quick_validate: 表格行总数仅从功能章节统计（_is_functional_section + _has_section_content） - 追踪具体缺失表格行的 (section, row, key, value) 信息 - _build_coverage_feedback: 反馈包含缺失表格行的具体内容 ## Impact | 指标 | 修复前 | 修复后 | |------|--------|--------| | 表格行分母 | 24 (含6行非功能) | 18 (仅功能章节) | | 表格行覆盖率 | 54.2% | 67% | ## Test - [x] pytest 全量通过 (102 passed, 13 skipped) Closes #21 [da-0601-0914]
pzhang_zywl added 1 commit 2026-06-01 14:04:30 +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>
pzhang_zywl closed this pull request 2026-06-02 14:11:50 +08:00
pzhang_zywl commented 2026-06-02 14:11:57 +08:00
Author
Owner
Copy Link
Copy Source

自行验证通过。变更已合入 main。

验证:

  • 表格行分母: 24 → 18 (仅功能章节) ✓
  • 表格行覆盖率: 54.2% → 67% → 72.2% ✓
  • _build_coverage_feedback 包含具体缺失行内容 ✓
  • test_layer_b_coverage PASSED ✓

相关 commit: 93e13e9 (已通过其他 PR 合入 main)
Stale PR #34 已关闭。

[da-0602-1321]

自行验证通过。变更已合入 main。 验证: - 表格行分母: 24 → 18 (仅功能章节) ✓ - 表格行覆盖率: 54.2% → 67% → 72.2% ✓ - _build_coverage_feedback 包含具体缺失行内容 ✓ - test_layer_b_coverage PASSED ✓ 相关 commit: 93e13e9 (已通过其他 PR 合入 main) Stale PR #34 已关闭。 --- [da-0602-1321]
Some checks are pending
CI / test (pull_request) Successful in 8s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
blocked
被其他 issue 阻塞，暂不处理
 product-code
产品功能相关
 test-code
QE-Agent 测试代码相关
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
dzhou_dev_agent_01 dzhou_qe_agent_01 hshen_dev_agent_01 hshen_qe_agent_01 phong_dev_agent_01 phong_qe_agent_01 pzhang_dev_agent_01 pzhang_qe_agent_01 pzhang_zywl vlu_dev_agent_01 vlu_qe_agent_01
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: zeekrAI/document_analyzer#34
Delete Branch "dev/issue-21-table-coverage-feedback"

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?