Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

REQ: Increase the ganged axis homing offset adjustment from +/- 2mm to +/- 10mm t support larger machines #169

Closed
5ocworkshop opened this issue Jul 28, 2022 · 1 comment

Comments

@5ocworkshop
Copy link

5ocworkshop commented Jul 28, 2022

I have a fairly large machine, my X is over 1100mm of travel. In going through detailed calibration after moving house and taking the machine apart, I discovered I needed to adjust my ganged Y axis homing offset to -5 mm to get back in to square. The firmware currently limits that value to +/- 2mm. It appears a lot of my error is derived from hand made sensor mount brackets.

It would be good to support a wider range of values, I suggest +/- 10.xxx mm for that variable to allow for errors that may exist in larger machines or and made machines/sensor brackets.

While there is the potential to rack the gantry if a high value is used here incorrectly, there are various ways a person can rack or damage a machine if they set incorrect values and this is largely a one-time calibration value that is not in the normal workflow so I can't think of any reason not to increase the allowable range.

core/settings.c

Line 508 in e3e7484

{ Setting_AxisAutoSquareOffset, Group_Axis0, "?-axis dual axis offset", "mm", Format_Decimal, "-0.000", "-2", "2", Setting_IsExtendedFn, set_axis_setting, get_float, is_setting_available },

 { Setting_AxisAutoSquareOffset, Group_Axis0, "?-axis dual axis offset", "mm", Format_Decimal, "-0.000", "-10", "10", Setting_IsExtendedFn, set_axis_setting, get_float, is_setting_available },

(Thanks to Andrew Marles for making me a quick build to test it).

@5ocworkshop
Copy link
Author

Thank you, Terje. You're a rock star.

In the end I needed 2.88 on my particular build, but this is a helpful change for larger format machines. Closing since it has completed and is in the release.

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

No branches or pull requests

1 participant