moved saveToolTable form iocontrol to tool_parse.cc, removed old tool…#415
moved saveToolTable form iocontrol to tool_parse.cc, removed old tool…#415rene-dev wants to merge 10 commits into
Conversation
…table file syntax, added pocketno to tooltable struct so non-random changers report the correct pocket number. should fix LinuxCNC#500 Signed-off-by: Rene Hopf <renehopf@mac.com>
Signed-off-by: Rene Hopf <renehopf@mac.com>
…git into tooltable-cleanup
Signed-off-by: Norbert Schechner <nieson@web.de>
Signed-off-by: Rene Hopf <renehopf@mac.com>
Signed-off-by: Rene Hopf <renehopf@mac.com>
Signed-off-by: Rene Hopf <renehopf@mac.com>
|
Still work in progress.... |
…t it Signed-off-by: Rene Hopf <renehopf@mac.com>
|
I am currently implementing a rack toolchanger wich is managed with pockets and M6 remap. Just wanted to comment that there are people out here with ATCs on LinuxCNC machines and this is an actual issue. |
|
It needs testing. the one bug in this PR is, that remapped T do not work for some reason. |
|
I have applied this PR to master, built it, and it now runs in our mill. It indeed fixes all issues and ATC now finally runs on Pockets. I will report if i suffer any additional bugs. I agree that T remap is unreasonable. Would rather have the pocket issue fixed and open this one instead. |
|
you are using a M6 remap only? |
|
Yes, no toolchange preparation whatsoever. Just move to tool location and pick up. |
|
ok, please check if all numbers match, including while loading-only and unloading-only a tool, by changing to and from tool 0. |
|
On 26 May 2018 at 18:46, Rene Hopf ***@***.***> wrote:
It needs testing. the one bug in this PR is, that remapped T do not work
for some reason.
but IMHO a T code should never be remapped,
Remapping T to use the normal T-only convention for a lathe tool-change
seems like one of the primary uses of remapping to me.
…--
atp
|
Signed-off-by: Rene Hopf <renehopf@mac.com> # Conflicts: # nc_files/remap_lib/python-stdglue/stdglue.py
|
Ah, thanks andy, that makes sense. I will look into why it doesn't work. |
|
I created a new pull request to 2.7: #528 |
…table file syntax, added pocketno to tooltable struct so non-random changers report the correct pocket number. should fix #500
Signed-off-by: Rene Hopf renehopf@mac.com