3.5B4: Input Shaping Plugin: Bug: Data collection cancelled

chrishamm

@alexjx I cannot confirm this, it works OK on my Duet 3 Mini5+ setup with 3.5-b4 which only has an accelerometer connected to the mainboard. The code you are referring to only captures the first number from the accelerometer string (0 -> 0, 1.0 -> 1) to identify the accelerometer to observe in the OM while a move is being performed.

It may help to check the JS console for potential errors if the data collection fails.

alexjx

@chrishamm I think the codes lead to a subtle situation where it just happened to work unintentionally. As far as I recall, duet mini 3 has CAN bus capability, by another word, there will be a valid CAN address.

Please see following picture, the accelerometerId is "0", so long if there is a board with canAddress equals to 0, the find will return a valid board. Therefore as long as we have a board with canAddress matching, we will not have the problem.

However with duet 2 wifi, we dont have such a board, it will always return undefined in line 495, and exception afterwards.

chrishamm

@alexjx Thanks, that's a valid concern. I just committed a possible bug fix to GitHub, are you able to test it?

tas

@chrishamm I can test this now on a Duet 2 Wifi in standalone. Can you provide a link to the file please, I can't find it. Must be too early in the morning.

droftarts

@tas Here is the commit https://github.com/Duet3D/DuetWebControl/commit/5db3108df0e8435e726b0ec5bc2954d09c422872

Ian

0

chrishamm committed to Duet3D/DuetWebControl

Potential bug fix for IS plugin

tas

@droftarts Many thanks but sorry I have no idea where to put those files. I was expect a zip of DWC. Any instructions appreciated.

alexjx

@tas I think you have to compile it. I will give it a try. I think it should work.

tas

@alexjx Thanks, sorry I wasn't able to help.

alexjx

@chrishamm The patch works.

droftarts

@chrishamm Can you post a patched DWC.zip?

Ian

jay_s_uk

@droftarts https://drive.google.com/drive/folders/1qd4OQXAwT2fin9BYjAkfLMPyyukXC77A?usp=drive_link

tas

@jay_s_uk Excellent! Thank you. This works on my Duet 2 Wifi standalone.