-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Clear the Alarm #36
Comments
I have implemented reading alarms in my fork, see https://github.com/ZdenekM/pydobot/blob/master/pydobot/dobot.py. There are |
ZdenekM, many thanks, I took a quick check in your fork. That alarm was quite helpful. I decided to download your code, used your example code and I get the result something like this, care to explain why I am getting this error message that the 'mode' isn't defined?
|
@SForney54 Which example code do you mean? I probably haven't touched https://github.com/ZdenekM/pydobot/blob/master/examples/basic.py at all. Try removing |
Hi, could this feature be merged to the main repo. Looks like move commands get sent to Dobot like normal, but the Dobot doesn't move if there are alarms about some axis being too close to limit. |
@jKostet Absolutely, that is the plan. I am helping luismesas with issues and maintenance and I would like to do a new release this month with some small improvements, like clearing the alarm and not crashing on empty messages (#44). Maybe this is the most straightforward thing to fix, so the next release could just include this. I still need to look at @SForney54's work myself. At this point in time I'm not sure whether the library should try to clear axis alarms itself to keep the arm moving, or if it should raise an exception and offer functionality to clear the alarm and retry the move. Sforney54 seem to clear the alarm in the constructor, i.e. clear alarms on connection to the robot. This is sensible, but what about during operations? |
Not sure but looking at the hardware I don't see physical end switches on the Dobot, so I think it just relies on absolute encoders, + the servo having a fourth cable for reading position. Thus I'd assume the limits are fixed in firmware and it should be safe Dobot-wise to actively ignore the alarms, e.g. having some variable that can be set in init like |
Apologies for jumping into this existing thread, after the fact. @ZdenekM I noticed that the command id for get_alarms and clear_alarms are both 20, yet the documentation, API and comms protocol for Dobot split them so that clear_alarms is 21. Is there a reason for your code using 20 for both? (PS I'm using Gustafsson's dobot-python repo, but the comms process is the same) |
Is there any way that can clear the alarm
Python code will not work unless resetting the Dobot Magician's current alarm
I don't think this library built-in with Alarm commands yet.
Thanks in advance
The text was updated successfully, but these errors were encountered: