-
-
Notifications
You must be signed in to change notification settings - Fork 551
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
Spindle Coolant Unexpected UI Behaviour #142
Comments
May you show me the result of your $G command? I just traced the code at https://github.com/gnea/grbl/blob/master/grbl/report.c#L321, I guess it may report M78 when both mist and flood are on. report_util_gcode_modes_M();
#ifdef ENABLE_M7
if (gc_state.modal.coolant) { // Note: Multiple coolant states may be active at the same time.
if (gc_state.modal.coolant & PL_COND_FLAG_COOLANT_MIST) { serial_write('7'); }
if (gc_state.modal.coolant & PL_COND_FLAG_COOLANT_FLOOD) { serial_write('8'); }
} else { serial_write('9'); }
#else
if (gc_state.modal.coolant) { serial_write('8'); }
else { serial_write('9'); }
#endif |
Hi @chamnit I uncommented ENABLE_M7 from config.h and found something strange. The $G parser state reported M78 when both M7 (Mist) and M8 (Flood) are turned on, may I ask if this is a desired behavior? Thanks. |
@cheton : It should say |
It looks great. Thank you for your prompt assistance. |
Awesome! |
CNCjs UI still needs work, if both M7 & M8 are on UI shows only M8 is on.
|
…mist coolant and flood coolant properly (#142)
Fixed in 1.9.0-alpha.5 |
The Coolant Buttons in the Spindle widget do not behave as expected.
I have two separate cooling / chip removal systems.
One is connected to GRBL's Flood Coolant, the other is connected to the GRBL's Mist Coolant.
I am able to enable/disable them independently or have them both on, which is good, I want the ability to have both on/off/or just one on.
The issue is that the UI buttons do not reflect the state of the Flood Coolant, and Mist Coolant properly.
If I enable Flood Coolant, then also enable Mist Coolant, both show on the UI as off.
They Flood Coolant and Mist Coolant should be able to be toggled independently of each other in the UI.
[cncjs@v1.9.0-alpha.4]
The text was updated successfully, but these errors were encountered: