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

[1.1.x] Simplify mesh bounds config #8043

Merged

Conversation

thinkyhead
Copy link
Member

@thinkyhead thinkyhead commented Oct 21, 2017

The default mesh bounds are based on mesh inset and bed size, with constraints applied based on physical bounds and probe offset. Only override if needed.

To simplify conditionals UBL_MESH_INSET is renamed MESH_INSET and UBL_MESH_(MIN|MAX)_[XY] can also drop the UBL_ prefix.

The conditional aspects of these settings need to move to Conditionals, as they already use IS_SCARA and other things not permitted in Configuration_adv.h anyway. The idea is that these settings can be left totally unset unless there's some pressing need to set them. And the default overrides are simple bed-size based. Craftier users may borrow elements of Conditionals_post.h.

For 1.1.x increment the configuration version numbers to "encourage" users to re-examine their configuration files.

@thinkyhead thinkyhead force-pushed the bf1_mesh_consolidate branch 3 times, most recently from 3278515 to 505557a Compare October 21, 2017 04:00
@thinkyhead thinkyhead changed the title Simplify mesh bounds config [1.1.x] Simplify mesh bounds config Oct 21, 2017
@thinkyhead thinkyhead force-pushed the bf1_mesh_consolidate branch 2 times, most recently from 8572bc5 to 22a3349 Compare October 21, 2017 04:48
@Roxy-3D
Copy link
Member

Roxy-3D commented Oct 24, 2017

This is a good thing getting everybody to a common name. I just hope it goes smoother than the X_BED_SIZE change. That broke a lot of stuff, especially with Delta's.

@thinkyhead
Copy link
Member Author

thinkyhead commented Oct 24, 2017

broke a lot of stuff

This adds no new options, so should be less disruptive.

@thinkyhead thinkyhead merged commit 209560f into MarlinFirmware:bugfix-1.1.x Oct 24, 2017
@thinkyhead thinkyhead deleted the bf1_mesh_consolidate branch October 24, 2017 23:56
kForth pushed a commit to kForth/Marlin that referenced this pull request Jun 22, 2022
…lidate

[1.1.x] Simplify mesh bounds config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants