[Emc-developers] M67

2011-09-05 Thread andy pugh
Someone mentioned on the forum that M67 doesn't work (he wanted to use it for controlling laser power) This is undeniably true. Pavel found a set of IRC logs on the subject showing that many have tried, but none have succeeded in making it work. So, Pavel had a go (patch at the end). It seems to wo

Re: [Emc-developers] M67

2011-09-06 Thread Pavel Shramov
On Tue, Sep 06, 2011 at 01:42:42AM +0100, andy pugh wrote: > I tried a 4-quadrant circle with an AIO change at 12, 3, 6, 9 'o'clock. > First, M68: Timing is good, but velocity stutters: > http://imagebin.org/171202 > Then with the M67 patch: > http://imagebin.org/171203 > The velocity still stutter

Re: [Emc-developers] M67

2011-09-06 Thread andy pugh
On 6 September 2011 13:06, Pavel Shramov wrote: > May you share gcode used for tests? >From memory (machine turned off, and miles away): G21 G0 X25.4 Y0 O100 REPEAT[100] F100 M67 E0 Q1 G2 X0 Y-25.4 I-25.4 M67 E0 Q2 G2 X-25.4 Y0 J25.4 M67 E0 Q3 G2 X0 Y25.4 I25.4 M67 E0 Q4 G2 X25.4 Y0 J-25.4 O100

Re: [Emc-developers] M67

2011-09-06 Thread Pavel Shramov
On Tue, Sep 06, 2011 at 01:28:23PM +0100, andy pugh wrote: > OK, I will try again tonight. I have had this before, and had to > make-clean to sort it out. Probably i've forgot to make clean first... Also my build is a bit truncated: ./configure --enable-run-in-place --enable-simulator --disable-gt

Re: [Emc-developers] M67

2011-09-06 Thread Pavel Shramov
On Tue, Sep 06, 2011 at 01:42:42AM +0100, andy pugh wrote: > There is an implicit limit of 32 of each channel as iomask_t is a u32. > This would need to change as current EMCMOT_MAX_DIO is 64 and > EMC_MOT_MAX_AIO is 16. (Both seem low to me) Fixed. Used unsigned long long (64 bits on most platform

Re: [Emc-developers] M67

2011-09-06 Thread andy pugh
Having pulled from the emc2-git (having noted that it has been pushed) I now get something that passes all the tests, and changes AIO without pausing motion. The small steps that were in my Halscope trace are there even with the M67 lines commented out, so are unrelated. -- atp "Torque wrenches a