Repository navigation
Lower Snapmaker 2.0 Bracing Kit Z acceleration to match base presets - #990
Open
highpowerxh wants to merge 2 commits into
Open
highpowerxh wants to merge 2 commits into
highpowerxh wants to merge 2 commits into
Conversation
Set A250/A350 inherited machine Z acceleration limits to 100 mm/s², including bracing-kit overrides. Leave the 40 mm/s Z speed limit and process travel settings unchanged. Investigated in Snapmaker#970. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Keep the existing 500 mm/s² base value and lower Bracing Kit overrides from 3000 to 500 pending hardware testing. User-directed revision of the preset-only fix for Snapmaker#989. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
This was referenced Oct 11, 2026
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
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.
Description
Closes #989. Related investigation: #970.
Lower
machine_max_acceleration_zfrom 3000 to 500 mm/s² in eight A250/A350 single/dual-head Bracing Kit base profiles, including Quick Swap + Bracing Kit variants.The shared
fdm_linear2base remains unchanged at 500. This revision matches the existing non-Bracing-Kit preset value instead of changing all Snapmaker 2.0 presets to the firmware default of 100. All 64 concrete A250/A350 presets now resolve to Z acceleration 500. Only the 32 Bracing Kit concrete presets change their effective Z acceleration.The reporter observed actual mechanical Z height falling behind the reported position. Lowering Z travel speed alone did not resolve it; excessive acceleration is the leading explanation. The Bracing Kit currently raises the base Z acceleration sixfold, without hardware validation located during this investigation. The reporter will test 500 next; 500 is a candidate inherited baseline, not yet a validated fix or a proven maximum reliable value. The absence of many reports does not itself establish reliability.
The official controller configuration uses Z maximum acceleration 100, which remains useful context but is not the value proposed in this revised PR. For idealized 0.4 mm rest-to-rest motion, acceleration 500 gives a peak of approximately 14.14 mm/s and duration 56.57 ms; acceleration 3000 gives 34.64 mm/s and 23.09 ms. Requests of either 40 or 100 mm/s are not reached. Actual acceleration may be lower due to
M204, junction planning or other firmware constraints; see #989 for the broader calculations.This is deliberately preset-only: keep the 40 mm/s Z speed ceiling, do not add
travel_speed_z=10, and exclude the separate writer/position-tracking changes from #970. Artisan, J1 and U1 definitions are unchanged. No dependency or schema changes. Existing user overrides may need manual adjustment; no forced migration is introduced.Screenshots/Recordings/Graphs
N/A: no UI implementation changes. Generated Bracing Kit machine-limit output changes from
M201 ... Z3000 ...toM201 ... Z500 ....Tests
git diff --check— passed.main:No application rebuild or native C++ suite was run specifically against this isolated preset-only revision. The prior 844-assertion result belongs to the separate experimental writer changes, not this PR. Physical testing with 500 is pending, including repeatability across affected variants.