5axes / Calibration-Shapes

A Cura plugin that adds simple shapes (cube, cylinder, tube) and also 24 Calibration and test parts + 7 Postprocessing scripts
GNU Affero General Public License v3.0
340 stars 69 forks source link

Acceleration Tower script using deprecated M204 syntax #92

Closed eSonOfAnder closed 2 years ago

eSonOfAnder commented 2 years ago

Noticed that the Acceleration Tower test did not show any changes to the print at the layer changes, and found that the script is adding the following lines:

M204 S400 M204 S450 etc.

Checking the Marlin Wiki, this syntax is deprecated, and is likely why this is not having any effect: https://marlinfw.org/docs/gcode/M204.html

5axes commented 2 years ago

Deprecated doesn't mean it's not working but just there is a new way to write this instruction. And M204 S is still working, but you should not activate the Acceleration management in Cura otherwise Cura will write is own M204 line and the instruction added by the script will be not operant. Note that Cura itself is using the format M204 S to manage the acceleration in the Gcode File : image

p-v-n commented 2 years ago

Possibly a dupe of https://github.com/5axes/Calibration-Shapes/issues/49

5axes commented 2 years ago

Yes it's the same discussion . And same conclusion for the moment.