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

G33 minor change #6621

Closed
wants to merge 6 commits into from
Closed

Conversation

LVD-AC
Copy link
Contributor

@LVD-AC LVD-AC commented May 7, 2017

-further named remaining tests
-A and O parameters obsolete
-quick homing with home_delta()
-preventing premature end

-further named remaining tests
-A and O parameters obsolete
-quick homing with home_delta()
-preventing premature end
@LVD-AC LVD-AC mentioned this pull request May 7, 2017
@@ -5082,7 +5078,7 @@ void home_all_axes() { gcode_G28(); }

const int8_t verbose_level = code_seen('V') ? code_value_byte() : 1;
if (!WITHIN(verbose_level, 0, 2)) {
SERIAL_PROTOCOLLNPGM("?(V)erbose level is implausible (0-2).");
SERIAL_PROTOCOLLNPGM("?(V)erbose Level is implausible (0-2).");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was changed in a previous merge, and it isn't important really, but it suggests a rebase may be needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I copied the whole G33 routine from a non-Git local copy on my drive I use to develop and beta test. Hence not 100% up to date with the latest merges....

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, no worries. Just better to not undo recent changes, if possible.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@bgort
Copy link
Contributor

bgort commented May 8, 2017

FWIW, it looks good to me.. but I don't have a delta so can't test.

@LVD-AC
Copy link
Contributor Author

LVD-AC commented May 8, 2017

tested and all is well

LVD-AC added 3 commits May 9, 2017 07:11
use the first probe result to prevent scraping the bed or floating to
high above it
@LVD-AC
Copy link
Contributor Author

LVD-AC commented May 12, 2017

Squashed into #6700

@LVD-AC LVD-AC closed this May 12, 2017
@LVD-AC LVD-AC deleted the G33-minor-update branch May 12, 2017 13:37
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