-
-
Notifications
You must be signed in to change notification settings - Fork 201
gcode_macro: Enable jinja2.ext.do extension #26
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
Conversation
|
I don't mind 👍 |
|
Should that not be at least mentioned in the documentation somewhere? Maybe even with the nice little danger marker if/once that gets approved? While I don't know what exactly that extension does I am pretty sure it would cause makros not to be directly portable to regular klipper. |
8277760 to
ca98cff
Compare
|
@chestwood96 it just adds support for a new |
|
I will add a section to the macro commands templates doc though |
ca98cff to
827a493
Compare
827a493 to
4d9a7f6
Compare
This adds a 'do' tag that works like a normal expression but ignores the return value. Useful for avoiding running the output of an expression as a command. Signed-off-by: Morten Lindhardt <r3fuze@gmail.com>
4d9a7f6 to
b9cff9d
Compare
1. gcode_macro: Enable jinja2.ext.do extension Klipper3d#26 KalicoCrew/kalico#26 2. gcode_macro: adds RELOAD_GCODE_MACROS Klipper3d#305 KalicoCrew/kalico#305 3. python gcode templates Klipper3d#360 KalicoCrew/kalico#360
1. expose math functions to gcode macros by mental Klipper3d#173 KalicoCrew/kalico#173 2. remove => gcode_macro: Enable jinja2.ext.do extension Klipper3d#26 KalicoCrew/kalico#26
1. gcode_macro: Enable jinja2.ext.do extension Klipper3d#26 KalicoCrew/kalico#26 2. gcode_macro: adds RELOAD_GCODE_MACROS Klipper3d#305 KalicoCrew/kalico#305 3. python gcode templates Klipper3d#360 KalicoCrew/kalico#360
1. expose math functions to gcode macros by mental Klipper3d#173 KalicoCrew/kalico#173 2. remove => gcode_macro: Enable jinja2.ext.do extension Klipper3d#26 KalicoCrew/kalico#26
original PR: Klipper3d/klipper#5149
@r3Fuze please let us know if you are against having your changes merged into danger-klipper!