Skip to content

Proposed fix for "FIXTHIS too verbose"#442

Merged
SebKuzminsky merged 1 commit intoLinuxCNC:masterfrom
KimK:master
Jun 15, 2018
Merged

Proposed fix for "FIXTHIS too verbose"#442
SebKuzminsky merged 1 commit intoLinuxCNC:masterfrom
KimK:master

Conversation

@KimK
Copy link
Collaborator

@KimK KimK commented Jun 15, 2018

I don't like the idea of having two different styles
for the G-code and M-code tables, but there are just
too many G-codes available to not have it be verbose.
So I ask the user to pick the blank cells for G-codes.

I could "reverse" the M-codes table too, if consistency
between them is thought to be important.

The tables are broken up to avoid trying to print
one table 100 lines long.

Signed-off-by: Kim Kirwan Kim@KimKirwan.com

I don't like the idea of having two different styles
for the G-code and M-code tables, but there are just
too many G-codes available to not have it be verbose.
So I ask the user to pick the blank cells for G-codes.

I could "reverse" the M-codes table too, if consistency
between them is thought to be important.

The tables are broken up to avoid trying to print
one table 100 lines long.

Signed-off-by: Kim Kirwan <Kim@KimKirwan.com>
@SebKuzminsky
Copy link
Collaborator

That looks great, thanks!

@SebKuzminsky SebKuzminsky merged commit 096cccb into LinuxCNC:master Jun 15, 2018
@andypugh
Copy link
Collaborator

71, 71.1, 72, 72.1 will be allocated at some point, but at the moment there is an existing remap for those who choose to install it, so those need to still be remappable.
Arguably even if/when they do get mainlined there is no reason to not add the remappable extras for those who prefer different behaviour.

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.

3 participants