-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Gmoccapy - G43 is disabled after selecting tool with zero z compensation #2613
Comments
Before selecting tool 0, G43 is active, after you select tool 0, G49 is active, i.e. tool compensation is off. |
So the question is why is G43 automatically disabled when tool 0 is selected? |
If a tool with Z=0 compensation is selected, G43 is deactivated. |
I did some research on this issue and found out that it is again a problem with the G43 auto-starting after a tool change. linuxcnc/src/emc/usr_intf/gmoccapy/gmoccapy.py Lines 3540 to 3544 in c2b5595
Norbert knows about this problem and plans to solve it. However, I will leave this Issue open so that we don't forget to check if his solution removes this consequence as well. My solution is:
|
I don't see how "tool offset == 0" can influence the code you cited. |
The cited code makes random problems. It makes race conditions. I have been dealing with this for a long time. lookt at: lookt at: lookt at: I would like ask you. Could you try reproduce this bug? I wouldn't be surprised if I was the only one with the problem. |
This issue may be related but I don't think this is caused by a race condition. |
In Axis you are changing the tools by MDI commands like T1 M6 and T2M6, but in gmoccapy you use M61 Q1 and M61 Q2. So that is one difference. I checked gmoccapy code and did not find any place, where M49 is commanded. So the problem may be deeper. Norbert |
RIP 2.9 Branche
Debian Bookworm
Oracle VM VirtualBox
Test sim Gmoccapy.
If any tool other than 0 is selected at the start, the z offset is displayed correctly.
If I choose tool 0, the z offset is displayed incorrectly.
The text was updated successfully, but these errors were encountered: