joakimtoe / KlipperLCD

A simple LCD service that enables the Neptune 3 Pro LCD screen with Klipper
Apache License 2.0
73 stars 11 forks source link

KlipperLCD requires max_accel_to_decel to be defined #11

Open Joldiges opened 6 months ago

Joldiges commented 6 months ago

AFAIK, max_accel_to_decel isn't enabled in configurations that don't have input-shaper (yet) enabled. KlipperLCD is requiring it to be defined. Perhaps have this script include use the dictionary default param to return -1 or N/A. (this is what I did for my own instance)

jofland commented 6 months ago

AFAIK, max_accel_to_decel isn't enabled in configurations that don't have input-shaper (yet) enabled. KlipperLCD is requiring it to be defined. Perhaps have this script include use the dictionary default param to return -1 or N/A. (this is what I did for my own instance)

Could you please post the code snippets of your workaround?

Joldiges commented 6 months ago

I'll raise a PR #12 is rasied

max_accel_to_decel has officially been deprecated

20240313: The max_accel_to_decel parameter in the [printer] config section has been deprecated. The ACCEL_TO_DECEL parameter of the SET_VELOCITY_LIMIT command has been deprecated. The printer.toolhead.max_accel_to_decel status has been removed. Use the minimum_cruise_ratio parameter instead. The deprecated features will be removed in the near future, and using them in the interim may result in subtly different behavior.