grbl-Mega icon indicating copy to clipboard operation
grbl-Mega copied to clipboard

Safety Door Input misbehavior

Open bdurbrow opened this issue 6 years ago • 6 comments

From a report by @zvizdic:

I tried using a safety door pin as my hold/resume since I have one on/off switch i place (see pics)but it’s not doing anything . Also try the feed hold and it works. I would rather have existing on/off switch doo hold and resume Any chance for some help Switch should be connected to control ground and pinD43 if I am right


OK, I think I found a problem with the safety door handling in the main grbl distribution. I've opened an issue over there; and patched my version here... it seems to be working at the moment.

There's something odd going on with updating the UI during a resume operation; I suspect that there's someplace that I need to drop a call to UITask() into that I haven't found yet.

Note that in order to resume operations, grbl expects both that the door switch has returned to normal; and that a cycle start has been issued (either on the serial port or on the cycle start input pin).

bdurbrow avatar Aug 10 '19 05:08 bdurbrow

I've adjusted the handling of the hold state... the UI updating seems to be working now.

bdurbrow avatar Aug 10 '19 07:08 bdurbrow

Ok where can I download latest software so i can test?

zvizdic avatar Aug 10 '19 13:08 zvizdic

All updates will be made to the main repository:

https://github.com/bdurbrow/grbl-Mega

Just click the green "Clone or Download" button; re-download the zip; and replace your old files with ones from the zip (obviously, if you made any changes - config.h or cpu_map.h files come to mind - you will want to keep the old files somewhere so you can re-apply those changes to the newly downloaded version).

😄

bdurbrow avatar Aug 10 '19 22:08 bdurbrow

Ok I got a file and tested it stopped but it doesn’t restart after ungrounded. Do I understand that right? If the pin is grounded it will stop ungrounded resume on its own.

zvizdic avatar Aug 12 '19 01:08 zvizdic

Press your cycle start button after returning the door switch to it's normal position.

bdurbrow avatar Aug 12 '19 16:08 bdurbrow

O OK Thanks

zvizdic avatar Aug 12 '19 21:08 zvizdic