Skip to content
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

Added zAxisAuto configuration option for spindle automation #458

Merged
merged 1 commit into from
Nov 27, 2017

Conversation

skiz
Copy link

@skiz skiz commented Nov 18, 2017

Add an option to the Settings panel which enables automated power control (M codes) to be handled by the AUX1 high signal. Accompanying firmware change is at MaslowCNC/Firmware#328

@BarbourSmith
Copy link
Member

How do we feel about putting this in the Advanced Settings? I feel like the less options we can give everyone up front the better (not that there aren't a bunch of other options in the settings which need to be cleaned up or moved to advanced settings)

@skiz
Copy link
Author

skiz commented Nov 20, 2017

Easy enough to change. Will do.

@BarbourSmith BarbourSmith merged commit 597e32d into MaslowCNC:master Nov 27, 2017
blurfl added a commit to blurfl/GroundControl that referenced this pull request Feb 16, 2018
PR MaslowCNC#458 added a setting to enable/disable spindle control. The setting was named 'zAxisAuto' which seems confusing as it is being used as an on/off control for spindle power. Change the name to 'spindleAutomation' to better reflect its use. Note that the change in GC will match an accompanying PR in Firmware.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants