Skip to content
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

TLO affects live plot in Axis #114

Closed
pkmcnc opened this issue Jul 19, 2016 · 4 comments
Closed

TLO affects live plot in Axis #114

pkmcnc opened this issue Jul 19, 2016 · 4 comments

Comments

@pkmcnc
Copy link
Contributor

pkmcnc commented Jul 19, 2016

Here are the steps I follow to reproduce the issue:

  1. Start a machine (axis_mm). Home.
  2. Edit TLO in the tooltable for a tool use Tcl_SetResult and Tcl_GetStringResult #1 to -200mm, save and reload the table.
  3. M6 T1 G43.
  4. G53 G0 Z0.
  5. Touch off Z to 0. Now both relative and machine Z = 0.

This is what I expected to happen:

The program preview should remain at Z=0 inside the limits box.

This is what happened instead:

The preview is far above the box.

It worked properly before this:

Probably never, in 2.7.5 and master it's as described.

@cradek
Copy link
Collaborator

cradek commented Oct 21, 2016

Thanks for this report - this is an old problem! I've fixed this and related issues in the commits ending at http://git.linuxcnc.org/?p=linuxcnc.git;a=commitdiff;h=5ef99c9 which will be in release 2.7.8. Please test when you can?

@pkmcnc
Copy link
Contributor Author

pkmcnc commented Oct 21, 2016

Thanks for the fix! I'll test asap.
Is it going to master too?

@jepler
Copy link
Contributor

jepler commented Oct 21, 2016

@pkmcnc yes it will be merged up to master. We "merge up" from release branches into master, just not always right away. It looks like in this case there's a conflict merging 2.7 to master due to some other changes by @SebKuzminsky -- when he gets those squared away, this will get merged too.

@pkmcnc
Copy link
Contributor Author

pkmcnc commented Oct 22, 2016

Just tested it in master, works as expected. Thank you!
I also observed some false "Program exceeds..." behavior from time to time, so I'm glad that this series of commits is supposed to fix it too. Thanks again!

@pkmcnc pkmcnc closed this as completed Oct 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants