interp: split loaded vs applied tool offset params (#5401-#5409, #5081-#5089)#4228
Open
grandixximo wants to merge 2 commits into
Open
interp: split loaded vs applied tool offset params (#5401-#5409, #5081-#5089)#4228grandixximo wants to merge 2 commits into
grandixximo wants to merge 2 commits into
Conversation
…1-#5089) Commit 725231f made #5401-#5409 report the tool length offset actually applied to motion. That matched the documentation, but broke probing routines in the wild that read #5401-#5409 right after an M6 (without a following G43) to get the loaded tool's stored offset. See LinuxCNC#4216. Restore the long-standing behavior of #5401-#5409 and put the applied offset in a new parameter set instead: - #5401-#5409: loaded tool's stored offset, refreshed from the tool table on tool change (M6), at startup, and by G10 L1/L10/L11. Reverts the convert_setup_tool / set_tool_parameters / default_tool_parameters / init_tool_parameters changes from 725231f so existing routines keep working. - #5081-#5089: tool length offset actually applied to motion, written by convert_tool_length_offset for G43, G43Hn, G43.1, G43.2, and zeroed by G49. This is the value 725231f intended to expose. Docs (overview.adoc, g-code.adoc) and the interp_parameter_def.hh enum updated to describe both sets. Fixes LinuxCNC#4216. Re-addresses LinuxCNC#2994 without the regression.
…semantics" This reverts commit 49ddfe6.
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
1 participant
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.
Forward-port of #4219 (merged to 2.9) to master, fixing #4216.
#5401-#5409 again hold the loaded tool's stored offset (refreshed on M6 / G10 / init), restoring the behavior probing routines in the wild rely on. The offset actually applied to motion by G43/G43.1/G43.2 moves to the new parameters #5081-#5089, and is zeroed by G49.
Two commits:
Full test suite passes locally (291 tests, 0 failed).