From da17f8ad37e5f8895f3db0e8f0cc5988556bafee Mon Sep 17 00:00:00 2001 From: Scott Lahteine Date: Mon, 12 Nov 2018 02:43:14 -0600 Subject: [PATCH] For now (!) don't synchronize in M204 and M205 See if this has any deleterious effect on motion planning by slicers that inser their own jerk and acceleration controls. The problem I see is that these settings can take effect on blocks already in the planner, and the only way around this would be to shadow these settings in planner blocks and use the shadowed settings in the planner. --- Marlin/src/gcode/config/M200-M205.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Marlin/src/gcode/config/M200-M205.cpp b/Marlin/src/gcode/config/M200-M205.cpp index 8b3562fb79..d30f7ba258 100644 --- a/Marlin/src/gcode/config/M200-M205.cpp +++ b/Marlin/src/gcode/config/M200-M205.cpp @@ -97,7 +97,7 @@ void GcodeSuite::M204() { SERIAL_ECHOLNPAIR(" T", planner.settings.travel_acceleration); } else { - planner.synchronize(); + //planner.synchronize(); // 'S' for legacy compatibility. Should NOT BE USED for new development if (parser.seenval('S')) planner.settings.travel_acceleration = planner.settings.acceleration = parser.value_linear_units(); if (parser.seenval('P')) planner.settings.acceleration = parser.value_linear_units(); @@ -131,7 +131,7 @@ void GcodeSuite::M205() { #endif if (!parser.seen("BST" J_PARAM XYZE_PARAM)) return; - planner.synchronize(); + //planner.synchronize(); if (parser.seen('B')) planner.settings.min_segment_time_us = parser.value_ulong(); if (parser.seen('S')) planner.settings.min_feedrate_mm_s = parser.value_linear_units(); if (parser.seen('T')) planner.settings.min_travel_feedrate_mm_s = parser.value_linear_units();