KevinOConnor

Thanks. However, the example configs need to follow the guidelines at https://www.klipper3d.org/Example_Configs.html . -Kevin

Thanks. What's the advantage of implementing this code as opposed to the user calling SET_INPUT_SHAPER in their `T0` and `T1` macros? -Kevin

> COPY and MIRROR IDEX modes that will, hopefully, eventually be merged into Klipper cannot be supported with Input Shaping using that approach, because both toolheads are active at the...

> First, AFAICT, there is presently a bug in hybrid_corex? implementation that makes input_shaper defunct on a toolhead change. And I saw some users complaining about this issue. I think...

> To offer another alternative for discussion myself that I considered and rejected initially, we could let users configure the input shaper on per-stepper basis I agree that does not...

The original documentation is correct; this PR would introduce an error to the documentation. The `printer.toolhead.extruder` returns a string, not an object that can be directly inspected. -Kevin

Thanks. I don't have a particular preference on the maximum neopixel chain size. However, at very large values it can result in command processing delays within the mcu and could...

Thanks. This does seem like useful functionality. Alas, I haven't gotten a chance to fully review it and I think it will be weeks before I can review it. Some...

> The two distinct positions for approach and departure must be used in most cases so a single rendezvous point would not be ideal. Okay, that makes sense. > 68%...

Thanks. Your description and pictures are very helpful. > The overall code in the module is centered around theses magnetic probes so there isn't much to strip out there. Okay....