Skip to content

Add element-box fusion and reading-order for set-of-marks#348

Merged
JE-Chen merged 1 commit into
devfrom
feat/element-parse-batch
Jun 22, 2026
Merged

Add element-box fusion and reading-order for set-of-marks#348
JE-Chen merged 1 commit into
devfrom
feat/element-parse-batch

Conversation

@JE-Chen

@JE-Chen JE-Chen commented Jun 22, 2026

Copy link
Copy Markdown
Member

set_of_marks.mark_elements numbers a single clean element list — but nothing produced it. A real screen parse yields three overlapping sources (OCR / icon / a11y boxes) with duplicates and no order. This is the missing connective tissue.

  • iou / merge_boxes (AC_… via fuse): IoU overlap of two boxes; drop near-duplicates above a threshold (largest kept).
  • fuse_elements (AC_fuse_elements): union OCR + icon + a11y boxes, tagging each with its source and dropping cross-source overlaps by source_priority (default a11y > ocr > icon, then larger area).
  • reading_order (AC_reading_order): band rows within row_tol px, order by x within each row, return new dicts with a stable index — ready to feed set_of_marks.
  • Plain dict boxes (x,y,width,height + extras) → pure-stdlib, fully headless-testable. Wired through all 5 layers + headless test + EN/Zh docs + WHATS_NEW. Qt-free.

@codacy-production

Copy link
Copy Markdown

Not up to standards ⛔

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@JE-Chen JE-Chen merged commit a28c95a into dev Jun 22, 2026
15 of 16 checks passed
@JE-Chen JE-Chen deleted the feat/element-parse-batch branch June 22, 2026 22:59
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant