KevinOConnor
KevinOConnor
@garethky - fyi. -Kevin
Thanks. In general, it seems fine to me. However, links to other documents should use relative links - replace `[Config_Reference](https://www.klipper3d.org/Config_Reference.html#common-kinematic-settings)` with `[Config_Reference](Config_Reference.md#common-kinematic-settings)`. -Kevin
Thanks for working on this. Is there a desire to merge this into the mainline Klipper branch? If so, what are the next steps for that. -Kevin
> I'd like to get it into Klipper Okay, let me know how you'd like to proceed with that. It also seems to me that it would be useful to...
Thanks. The biggest issue on my side is time constraints. If you can point me to some core areas of code that you'd like me to take a look at,...
Thanks. As high-level comments, I think we'd want one driver in the mainline code. It'd also be generally preferable to simultaneously support python2/3 (as it's typically relatively simple to extend...
> For python2.. I think it would be difficult. There's e.g. a lot of type hints on the code, as well as I don't know what else that's python 3...
Thanks for working on this. I got a chance to review the code in the repo mentioned above. I have some initial comments. My main feedback is that the new...
Thanks. I agree we'll need to change the code to support the latest version of the sdk. Unfortunately, it looks like `pio_sm_hw_t` was introduced in v1.3.0 of the SDK, and...
Thanks. I'll look to merge #57 in a couple of days. -Kevin