Apply full TCP transforms through planning and readback - #43
Draft
Jepson2k wants to merge 1 commit into
Draft
Conversation
This was referenced Sep 7, 2026
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.
Implements queued full TCP correction and readback across the controller, planner, FK/IK, client and preview. A program can set translation and intrinsic XYZ orientation, then confirm the applied value before using it. Earlier queued motion retains its frame, cancelled pending corrections do not leak into planning, and a tool/variant change clears the correction.
The XYZ API remains available and clears user rotation. Both TCP queries now fail explicitly when the controller does not answer. Tool status includes its applied variant, with legacy status decoding preserved, so a calibration cannot silently bind to different jaws.
Validation: real fake-serial workflows cover full-transform readback, stationary status, local FK, rotated TRF motion, preview, cancellation, legacy XYZ behavior, variant changes and hostile wire values. All 42 selected regression tests pass, including protocol compatibility and existing motion/tool workflows; pre-commit passes.
Depends on #42 and the companion waldoctl contract PR. Uses waldoctl v0.15.0; release the contracts first, then PAROL6 v0.9.0, before Commander adopts those release pins. Same branch name across the companion PRs enables branch-matched CI.
Companion PRs: waldoctl, PAR6, Commander.