klipper icon indicating copy to clipboard operation
klipper copied to clipboard

Update mcu.py to reduce Can Bus false timeouts

Open tinymachines3d opened this issue 3 years ago • 3 comments

Increase timeout amount to avoid false timeouts with multiple mcus. ip -s link show can0 reports zero bytes, packets lost.

tinymachines3d avatar Oct 08 '22 00:10 tinymachines3d

This change would increase the amount of "homing overshoot" that could result - and thus would conflict with the information in docs/Multi_MCU_Homing.md . In general, I'd like to see an analysis of the root cause of the problem, before changing the code; otherwise "fixing symptoms" often results in unanticipated regressions.

Separately, commits need to have a signed-off-by line - see https://www.klipper3d.org/CONTRIBUTING.html .

-Kevin

KevinOConnor avatar Oct 31 '22 14:10 KevinOConnor

Hi @KevinOConnor @tinymachines3d I can confirm this is happening on my setup as well and I'm not sure how to debug it: I have 1Mbaud can0 running well (albeit from the outputs of ip show link can0 -detail it seems there are scarce errors) I have even compiled RT kernel for rpi and set real-time priorities and affinities for klipper to limit the possibility it's caused by something else stalling klipper of cycles when needed and it still occurs). Any suggestion for getting more useful debug info and/or what else I should experiment with or try?

jtrmal avatar Nov 21 '22 22:11 jtrmal

Any suggestion for getting more useful debug info and/or what else I should experiment with or try?

Open a new topic on Klipper Discourse and be sure to attach the full Klipper log running the pristine Klipper code.

-Kevin

KevinOConnor avatar Nov 22 '22 01:11 KevinOConnor

It looks like this GitHub Pull Request has become inactive. If there are any further updates, you can add a comment here or open a new ticket.

Best regards, ~ Your friendly GitIssueBot

PS: I'm just an automated script, not a human being.

github-actions[bot] avatar Dec 13 '22 12:12 github-actions[bot]