search

grblHAL / core

grblHAL core code and master Wiki
Other
304 stars 73 forks source link

Lathe Diameter/Radius offset report issue #500

Closed nickshl closed 3 weeks ago

nickshl commented 2 months ago

When G7/G8 is send, X DRO changes from Radius/Diameter to some "junk" value, then it changed to correct Radius/Diameter value. I suspect it happens because WCO does not reported with same report as Radius/Diameter mode:

<Idle|MPos:4.756,0.000,0.000|Bf:100,1024|FS:0,0|WCO:7.800,0.003,0.000> G7 ok <Idle|MPos:9.512,0.000,0.000|Bf:100,1024|FS:0,0|D:1> <Idle|MPos:9.512,0.000,0.000|Bf:100,1024|FS:0,0> <Idle|MPos:9.512,0.000,0.000|Bf:100,1024|FS:0,0|WCO:15.600,0.003,0.000>

So, between report contains D:1 and last line, old offset is used to calculate X axis position on sender side. Not a terrible bug, but small annoyance. I wish correct WCO can be reported same line as mode, something like that:

<Idle|MPos:4.756,0.000,0.000|Bf:100,1024|FS:0,0|WCO:7.800,0.003,0.000> G7 ok <Idle|MPos:9.512,0.000,0.000|Bf:100,1024|FS:0,0|D:1|WCO:7.800,0.003,0.000>

nickshl commented 3 weeks ago

Works! Thanks.