-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Remove legacy Auto Bed Leveling code #4749
Conversation
All values in bytes. Δ Delta to base
|
if (cs.add_homing[Z_AXIS] < 0) negative_z_offset = negative_z_offset + cs.add_homing[Z_AXIS]; | ||
#endif | ||
if (target[Z_AXIS] < min_pos[Z_AXIS]+negative_z_offset) target[Z_AXIS] = min_pos[Z_AXIS]+negative_z_offset; | ||
if (target[Z_AXIS] < min_pos[Z_AXIS]) target[Z_AXIS] = min_pos[Z_AXIS]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you're wondering where 14 bytes of flash are saved. It is here, by removing negative_z_offset
since it is always 0.
f2b3ff4
to
93d7e4b
Compare
The goal is to clean up the project
93d7e4b
to
2b1c49d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved, but see some further cleanup I noted above.
We discussed removing this code some time ago in 2021 (See #3227) and I want to propose again to remove auto bed leveling. The code has been unmaintained for many years and doesn't compile. This functionality has also not been tested / verified for many years.
The goal is to clean up the project so it's easier to maintain and further improve the project.