Error in tool_offsetview.py

More
28 Jul 2024 21:07 #306279 by cmorley
Replied by cmorley on topic Error in tool_offsetview.py
Yes spindle stop button doesn't use m5 it calls spindle off command.
Either way should call the spindle off command.

It will be something in the code that assumes that S 0 = spindle stop, which is not true anymore.

M70/M72 had something similar happen in the right situation.
IIRC it had something to do with using MDI to turn on the spindle and the stop button to turn it off then the M72 would turn it back on because it never got the M5 command.
It should be checking if the spindle is on/off rather then the Gcode M3/4/5.

Your situation sounds the same. I just don't remember if we worked around it or not.

Chris

Please Log in or Create an account to join the conversation.

More
28 Jul 2024 21:44 #306283 by MarkoPolo
Oh, I wrote that I don't use m70/72. I looked through my ngc again and it does indeed have m70 and m72.

Putting m5 at the beginning of the code solves the problem, maybe in qtvcp procedures such a small correction would also be enough..

Please Log in or Create an account to join the conversation.

More
29 Jul 2024 02:42 #306302 by cmorley
Replied by cmorley on topic Error in tool_offsetview.py
Well that masks the problem, and in some probing the spindle should be on.
But I agree we need to protect the user in the mean time.

This is one of those deep seeded, need to look at the big picture problems.
We aren't very good at those :)

Please Log in or Create an account to join the conversation.

More
29 Jul 2024 03:19 #306303 by cmorley
Replied by cmorley on topic Error in tool_offsetview.py
I can make it happen by:
MDI: M3 S100
manual stop spindle
MDI: M70
MDI: M72

Spindle starts.
The following user(s) said Thank You: tommylight

Please Log in or Create an account to join the conversation.

More
30 Jul 2024 19:49 #306435 by MarkoPolo
For now, m5 solves the problem in my tool measurement procedure. I do not use the built-in qtvcp procedures. Let someone smarter figure out how to solve the problem, I probably can't help.

Returning to the topic of the axis buttons function. Could you add the "go to zero" function?.
I only added the titles to the axis_tool_button.py file to the screenshot. I do not have the functions working yet.
Attachments:

Please Log in or Create an account to join the conversation.

More
03 Aug 2024 16:34 #306864 by cmorley
Replied by cmorley on topic Error in tool_offsetview.py
yes.
I also have done some work on having the DRO clickable with similar options.
I think I will set it up as stylesheet options, so a user can change them fairly easily.
The following user(s) said Thank You: tommylight, MarkoPolo

Please Log in or Create an account to join the conversation.

More
03 Aug 2024 23:49 #306889 by cmorley
Replied by cmorley on topic Error in tool_offsetview.py
Ok that work is pushed to master now.
I have not updated the docs yet, but stylesheets can change the options.
The menu items in the DRO or axis selection buttons can be made the same or any combination of those options.
Also now you can recover from pressing the zero button by selecting 'set to last' in the dro widget menu options.

let me know what you think.

Chrs

Please Log in or Create an account to join the conversation.

More
04 Aug 2024 13:03 #306913 by MarkoPolo
I tested it a bit, it works very well.
I like it.
It could be everything even in one menu, axis button, but if it is possible to customize the content of both menus, it is very good.
How to do it in the style sheet?
In qtdragon_hd, the ref buttons are probably no longer needed? If so, I can customize the layout..

Marek

Please Log in or Create an account to join the conversation.

More
04 Aug 2024 14:44 - 04 Aug 2024 14:46 #306922 by cmorley
Replied by cmorley on topic Error in tool_offsetview.py
Here are the stylesheet properties, you can turn them on and off as you please.
They will change immediately.
DROLabel{
qproperty-showLast: true;
qproperty-showDivide : true;
qproperty-showGotoOrigin: true;
qproperty-showZeroOrigin: true;
qproperty-showSetOrigin: true;
}

 AxisToolButton{
qproperty-showLast: false;
qproperty-showDivide : false;
qproperty-showGotoOrigin: false;
qproperty-showZeroOrigin: false;
qproperty-showSetOrigin: false;
}
Last edit: 04 Aug 2024 14:46 by cmorley.

Please Log in or Create an account to join the conversation.

More
04 Aug 2024 15:10 #306925 by MarkoPolo
Great.
That all works.
You could also add "dro_axis_*" to the "button_response_list"
in the handler file so that these functions are not available during operation.

Please Log in or Create an account to join the conversation.

Moderators: cmorley
Time to create page: 0.267 seconds
Powered by Kunena Forum