Skip to content

Offset Toolsetting#124

Open
benagricola wants to merge 7 commits into
mainfrom
toolsettingoptions
Open

Offset Toolsetting#124
benagricola wants to merge 7 commits into
mainfrom
toolsettingoptions

Conversation

@benagricola

Copy link
Copy Markdown
Member

Building on top of #96, this removes the manual approach (I'm not happy with the safety considerations of this, yet), and changes some of the messaging to be more consistent with the rest of MillenniumOS.

Not tested yet but no logic changes in here.

@benagricola
benagricola requested a review from bdistin August 27, 2024 08:36
@bdistin

bdistin commented Aug 29, 2024

Copy link
Copy Markdown
Collaborator

The changes you've made are fine. But even though I have been using this, (when my machine was operational), I'm not completely convinced it's the best answer. Say if one blade is lower or higher than the rest, you will get an effectively wrong z offset.

My current musing, although without any code or testing behind it, is instead of asking which type of probing routine it's all automatic radial, however, you are asked center/left/right/front/back instead. "Center" is default and exactly how automatic radial works now. The other directions will offset to start with and pause for you to align the tool like as in this pr. However, instead of only taking one measurement, after the first dive, it will continue checking around the circumference of what's within the work envelope. Would also implement a limit based on that first measurement, like automatic radial works now, which probably addresses the safety concerns you have, if I were to guess.

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.

2 participants