Stereo Import menu: hide registration section, align Warp to All, interactive-stereo warp#1787
Open
mattdawkins wants to merge 3 commits into
Open
Stereo Import menu: hide registration section, align Warp to All, interactive-stereo warp#1787mattdawkins wants to merge 3 commits into
mattdawkins wants to merge 3 commits into
Conversation
Per-camera registration import is meant for multicam rigs; stereo datasets use the calibration (camera file) import instead, so the Import menu no longer shows the registration buttons when the dataset subtype is stereo. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The Import menu's "Warp to All" checkbox now lines up with the Overwrite checkbox instead of sitting partially below it. On stereo datasets the option now works through the interactive stereo service: when interactive stereo is enabled and a camera calibration is present, checking it warps every imported detection from the active camera to the other one (head/tail lines as lines, which also computes the stereo measurement; polygons as polygons; plain boxes as boxes), skipping frames the other camera already has. When interactive stereo is off or no calibration is loaded the option is greyed out, with the hint explaining what is missing. Multi-camera (non-stereo) datasets keep the registration-based warp. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The persistent hint under the checkbox stretched it and wrapped the row, leaving Warp to All below Overwrite. The two checkboxes now sit side by side in a no-wrap row (dense, no reserved detail space) and the availability hint moved into a hover tooltip. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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
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.
Two related Import-menu changes for stereo datasets:
1. Hide the Import Registration section on stereo datasets
Per-camera registration transform import is meant for (non-stereo) multicam rigs; stereo rigs use the calibration (camera file) import instead.
registrationSupportednow also requires the dataset subtype not bestereo.2. "Warp to All" alignment + interactive-stereo behavior
🤖 Generated with Claude Code