Back
[00:50:52] <dgarr> cradek: a note/patch and a question on random_toolchanger with non-random_toolchanger config:
http://www.panix.com/~dgarrett/stuff/rtc_note.txt
[00:55:04] <cradek> dgarr: won't the last two blobs in the patch break random? you'll get the pocket twice and never the toolno.
[00:56:31] <cradek> previously I thought the columns were [don't care] then toolno. you are right that they were toolno then [don't care]
[00:57:10] <cradek> maybe random should be toolno THEN pocket
[00:57:23] <cradek> nonrandom (when writing) should be toolno, toolno
[00:57:31] <cradek> nonrandom (when reading) should be toolno, [ignore]
[00:58:15] <cradek> I bet TnM6-twice that you found is a bug
[01:01:44] <cradek> when writing the tool table on nonrandom, we could write out toolno,toolno or toolno,0
[01:02:19] <cradek> for some of history, emc was broken if toolno and fms didn't match. that's why I figured I'd write them out as both toolno, but that's a pretty weak reason.
[01:03:28] <cradek> heck I could even save the fms just like the comment. it would only be a little bit silly.
[01:05:16] <dgarr> oops , on the last blobs imeant
[01:05:21] <dgarr> - toolTable[pocket].toolno,
[01:05:21] <dgarr> + random_toolchanger? toolno: toolTable[pocket].toolno,
[01:07:53] <dgarr> i agree with16:57:23,31
[01:08:07] <cradek> I have a new patch for you to review - one minute
[01:12:01] <cradek> http://timeguy.com/cradek-files/emc/0001-preserve-fms-values-for-nonrandom-tc.patch
[01:12:37] <cradek> ... untested
[01:13:31] <cradek> yuck, fakepocket can overrun
[01:16:34] <cradek> but aside from that?
[01:32:19] <cradek> http://timeguy.com/cradek-files/emc/0001-preserve-fms-values-for-nonrandom-tc-and-more.patch
[01:32:29] <cradek> ^ obvious problems removed
[01:35:15] <dgarr> i tested your first patch in sim, read files ok, is the second to be applied sequentially or start over?
[01:36:53] <cradek> start over, sorry
[01:37:09] <cradek> just added some missing error handling
[01:44:28] <dgarr> tested reading tbl with sim, ok when is saveToolTable used?
[01:45:00] <cradek> G10 L1, tool touch off, and whenever changing tools if random
[01:52:09] <dgarr> write seems ok (mdi g10) too
[01:53:12] <dgarr> i ran across these items working on numbered parms for tool items, but the repetition with tnm6 is beyond my ken
[01:53:48] <cradek> I'll have to go back and see where that was previously filtered
[01:54:19] <cradek> because pockets are prepped, you can't just throw out the tool change if the pocket hasn't changed - on random, switching between the same two tools over and over never moves the carousel
[01:54:57] <cradek> thanks so much for testing it. I'm too close to see the problems.
[02:00:48] <dgarr> np
[02:01:53] <CIA-3> EMC: 03cradek 07master * rf22ab22058a1 10/configs/ (43 files in 37 dirs): fix up all the tool tables, yet again
[02:02:04] <CIA-3> EMC: 03cradek 07master * rbb0e922fef39 10/src/emc/iotask/ioControl.cc: preserve fms values for nonrandom tc, and more
[02:09:28] <cradek> now I must remember to change my tool table around, or I'll have a mess
[14:53:14] <CIA-3> EMC: 03jthornton 07v2_3_branch * rcc53a58345bc 10/docs/src/gcode/ (5 files in 2 dirs): Add diagrams to help explain G2/3 moves
[14:53:15] <CIA-3> EMC: 03jthornton 07v2_3_branch * r5fc94061a521 10/docs/src/hal/drivers.lyx: Fix link
[17:03:53] <CIA-3> EMC: 03jthornton 07master * r3b76e46e1ee1 10/docs/src/gcode/ (5 files in 2 dirs): Add example for G2/3
[17:04:16] <CIA-3> EMC: 03jthornton 07v2_3_branch * re6942710a8c3 10/docs/src/gcode/main.lyx: grammar
[20:19:44] <cradek> jepler: setting the probe to 2ms (from 0) really improved it