Skip to content

Keep every mask component in point-click segmentation - #1952

Open
mattdawkins wants to merge 1 commit into
mainfrom
dev/point-seg-multipolygon
Open

mattdawkins wants to merge 1 commit into
mainfrom
dev/point-seg-multipolygon

Conversation

@mattdawkins

Copy link
Copy Markdown
Member
  • Point-click segmentation only kept the largest exterior of the mask; it now stores every component the service returns, holes included, under SegmentationPolygon, SegmentationPolygon-1, ... (the same keys auto-populate uses in Auto-populate a new box or line with a segmentation polygon, head/tail points or a tighter box #1941)
  • Preview, multi-frame confirm, frame-navigation restore and the polygon-edit takeover all carry the full list; a refinement removes the components the new mask no longer has, and bounds cover all of them
  • Reset already removes any segmentation-added key, so it is unchanged
  • New segmentationPolygons.ts helper shared by the recipe and the mode manager; specs for the helper, the recipe and the mode manager
  • Stereo: the other-camera copy still receives the service's single polygon; extending the stereo-segment response is a separate VIAME change

🤖 Generated with Claude Code

The service's polygons list, with holes, is stored as keyed polygons on the
detection; a refinement drops the components it no longer has.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
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