fix(geometry): separate real line contact from diagonal bbox overlap (#234) - #235
Merged
Merged
Conversation
moonweave
force-pushed
the
agent/issue-234-line-iou-overreport
branch
from
August 3, 2026 00:47
7385539 to
8a7aa5c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #234.
변경 내용
artist_pair_iou는 기존 호환성을 위해 유지하되, 대각선 선분의 축평행 bbox가 만드는 오탐을 실제 선 접촉과 구분할 수 있도록 정확한 중심선 증거를 추가했습니다.centerline_intersection_px와centerline_intersects를 추가iou와artist_overlaps판정은 변경하지 않음axhline/axvline등 blended transform과 비기본 축 범위를 검증근본 원인
대각선 선분의 bbox는 실제 선보다 넓기 때문에, 선에 닿지 않는 범례도 bbox 겹침으로 보고될 수 있었습니다. 또한
line_text_crossings는 범례를 검사하지 않으므로 0은 비충돌 증거가 아니라 검사 범위 밖이라는 뜻이었습니다.검증
정책 임계값이나 severity는 새 기하학적 사실에 포함하지 않아 기존 소비자와의 호환성을 유지합니다.