-
Notifications
You must be signed in to change notification settings - Fork 94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CNCjs Issue due to bug in mode state reporting #33
Comments
that's present in the usbcnc code as well, and I verified that removing the line fixes the issue (still using the usbcnc version because this one shuffled pins around and I already spent 2 days making the damn thing work on veroboard, I'm sick of soldering for now :D ) |
Yes, found the same bug in the forked usbcnc's repo. Anyway, 6-AXIS-USBCNC_GRBL is an excellent piece of work and runs my brand new CNC extremely well. A five grands machine with a $1.99 brain ;-) |
I'll try to fix it. |
just remove / comment the line, it fixes the issue |
done |
CNCjs is not able to parse modal state reports by 6-AXIS-USBCNC-GRBL due to a small glitch in report.c. Cause is an additional
report_util_gcode_modes_M();
in Line 318 of report.c here which is not in the original grbl v1.1fIt creates this (wrong syntax)
[GC:G0 G54 G17 G21 G90 G94 M0 M M9 T0 S0 F0]
instead of this (correct syntax)
[GC:G0 G54 G17 G21 G90 G94 M0 M9 T0 S0 F0]
This Bug will cause GrblLineParserResultParserState.js to ditch all modal reports and not update its grbl info panels accordingly:
The text was updated successfully, but these errors were encountered: