setting up schaublin 125 cnc lathe with carousel toolchanger

More
7 years 8 months ago #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 to join the conversation.

More
7 years 8 months ago #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

Please Log in to join the conversation.

More
7 years 8 months ago #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

Please Log in to join the conversation.

More
7 years 8 months ago #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 to join the conversation.

More
7 years 7 months ago #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 to join the conversation.

More
6 years 1 week ago #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

Please Log in to join the conversation.

More
6 years 1 week ago #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

Please Log in to join the conversation.

More
6 years 1 week ago #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.

Please Log in to join the conversation.

More
6 years 1 week ago #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 to join the conversation.

More
6 years 1 week ago #186731 by andypugh
The PR should be against 2.8 as this is a bug.

Please Log in to join the conversation.

Time to create page: 0.064 seconds
Powered by Kunena Forum