setting up schaublin 125 cnc lathe with carousel toolchanger

More
02 Feb 2019 21:47 #125561 by trilobyte
Hi Andy

i created the carousel.comp with notepad++ on windows and copied the file via win scp to the linuxcomputer. after that the halcompile worked.

Unfortunately position 0 still does not work.

i edited the original carousel.comp with this:

"case 'S': // individual sensors
for (i = inst_sense - 1; sense(i) == 0 && i > 0 ; i--) {}
if (sense(i)) p = i + 1;
break;"

enable is true, motor fwd isn't.
position 1,2,3 are working.

peter

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

More
05 Feb 2019 09:27 - 05 Feb 2019 11:11 #125721 by inoxix
Hi Andy

I created a fix for Peter and he confirmed it's now working. I will create a pull request as soon as I have figured out all the github stuff ;).

We also found another glitch in the code when T0 is requested. The carousel goes into an endless mode. In case of turret tool changer T0 doesn't make much sense and an error handling could be added. Maybe this is by intention and a useful case in other configurations.

Tom

Update:
Pull Request created: github.com/LinuxCNC/linuxcnc/pull/557
Last edit: 05 Feb 2019 11:11 by inoxix.

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

More
06 Feb 2019 16:28 - 06 Feb 2019 16:29 #125820 by andypugh
Sorry for the late reply, I am on a skiing holiday. I did manage to try the fix on my pocket computer, but for some reason I can’t even set “single” mode on that, the strncmp lines all fail.
Thanks for confirming that the fix works. Was the problem halcompile rather than halcompile —install?

The pull request contains quite a lot of spurious white-space changes. I would prefer it if it didn’t.

T0 means unload tool on milling machines. Not very applicable to lathes and probably best handled in the handler code. But it sounds like you have the system using tool number and not pocket number. T0 probably shouldn’t ask for pocket 0
Last edit: 06 Feb 2019 16:29 by andypugh.

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

More
07 Feb 2019 10:10 #125913 by inoxix
Enjoy your holidays!

I updated the branch just to contain the needed change.
However, there are some tabs (lines 301, 305, 316) and indention errors (lines 181, 305). I also saw that the use of break statement and curly brackets is not really consistent.

For T0 I agree, on mils T0 has different meaning. Therefore, I would also like it is.

Regards
Tom

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

More
06 Mar 2019 09:42 #127873 by trilobyte
after apt-get update and apt-get dist-upgrade is the machine now working with the carousel.comp!

Regards
peter

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

More
18 Oct 2020 13:02 - 18 Oct 2020 13:04 #186461 by trilobyte
i used the machine with 4 tools without problems. but i have a problem with more than 4 tools.

i have 4 pockets. if i try to switch to tool #5 the carousel active bit gets high, but the fwd is still low.

i load the carousel with this line in hal:

loadrt carousel pockets=4 encoding=single num_sense=4 dir=1

i tought i can call tool #5 and the carousel.comp calculates 5%4 = 1 (tool #5 in pocket 1)
this isn't working.

with 4 tools it works well, but if i use a tool# higher dann 4 it will not work.

does anyone have an idea what i could do?

peter

ps: i use the latest 2.8
Last edit: 18 Oct 2020 13:04 by trilobyte.

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

More
19 Oct 2020 20:59 - 19 Oct 2020 21:07 #186621 by trilobyte
i made a short video:

Video Schaublin 125

and a screenshof of the halshow

www.trilobyte.ch/temp/20201019_224004.jpg


does anyone have any idea what else I could try?

peter
Last edit: 19 Oct 2020 21:07 by trilobyte.

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

More
20 Oct 2020 10:57 - 20 Oct 2020 13:15 #186668 by andypugh
I think this is a bug. Look at:
github.com/LinuxCNC/linuxcnc/blob/2.8/sr...s/carousel.comp#L261

It almost certainly should be testing mod_pocket not not pocket_number there.

Do you want to test it? It should be possible to download and edit that file, then compile/install it with "sudo halcompile --install carousel.comp"
You might need to install the "linuxcnc-dev" package, but you might already have it.
Last edit: 20 Oct 2020 13:15 by andypugh.

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

More
20 Oct 2020 18:48 #186724 by inoxix
I checked github.com/LinuxCNC/linuxcnc/blob/2.8/sr...s/carousel.comp#L224 and found it does what I would expect but overlooked the usage below. Now it seams obvious.

I will do the change and test with Peter. If ok I create a PR for this fix.

BR
Tom

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

More
20 Oct 2020 19:29 #186731 by andypugh
The PR should be against 2.8 as this is a bug.

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

Time to create page: 0.141 seconds
Powered by Kunena Forum