Skip to content

fix: handle Qwen and Gemini grounding coordinate orders - #25

Merged
Anionex merged 2 commits into
mainfrom
fix/portable-bbox-schema
Aug 17, 2026
Merged

fix: handle Qwen and Gemini grounding coordinate orders#25
Anionex merged 2 commits into
mainfrom
fix/portable-bbox-schema

Conversation

@Anionex

@Anionex Anionex commented Aug 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • select Gemini yxyx versus Qwen-family xyxy grounding coordinates by model family, with VISION_BOX_ORDER override support
  • reject truncated bounding-box JSON instead of returning partial matches
  • make detect target construction idempotent
  • document the provider coordinate conventions in the vision-tools Skill

Verification

  • python3 tests/test_ground.py
  • python3 tests/test_detect.py
  • python3 -m py_compile ground.py detect.py
  • live Qwen3.6 grounding on a 1200x675 image produced a horizontal top-title box after xyxy parsing

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@Anionex
Anionex merged commit b4c31d6 into main Aug 17, 2026
4 checks passed
@Anionex
Anionex deleted the fix/portable-bbox-schema branch August 17, 2026 00:40
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