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.

Sign up for GitHub

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

Jump to bottom

Set homing feedrates individually #4358

Merged
merged 2 commits into from
Jul 20, 2016
Merged

Set homing feedrates individually #4358

merged 2 commits into from
Jul 20, 2016

Conversation

thinkyhead
Copy link
Member

@thinkyhead thinkyhead commented Jul 20, 2016 •
edited
Loading

So we can use the configured Z homing rate as the configured Z probing rate, if desired.

  • Add HOMING_FEEDRATE_XY and HOMING_FEEDRATE_Z
  • Get rid of HOMING_FEEDRATE_XYZ in Delta configurations
  • Use only HOMING_FEEDRATE_Z in Delta configurations
  • Drop HOMING_FEEDRATE altogether, sanity check for its existence

@thinkyhead thinkyhead force-pushed the rc_individual_homing_rates branch from 64d413b to 71a3696 Compare July 20, 2016 00:21
@thinkyhead thinkyhead force-pushed the rc_individual_homing_rates branch from e88af95 to e44b37c Compare July 20, 2016 00:32
@jbrazio jbrazio modified the milestone: 1.1.0 Jul 20, 2016
@thinkyhead thinkyhead merged commit 792fff5 into MarlinFirmware:RCBugFix Jul 20, 2016
@thinkyhead thinkyhead deleted the rc_individual_homing_rates branch July 20, 2016 01:20
@thinkyhead thinkyhead mentioned this pull request Jul 25, 2016
Closed
drewmoseley pushed a commit to drewmoseley/Marlin that referenced this pull request Nov 8, 2023
@3d-gussner
Merge pull request MarlinFirmware#4358 from 3d-gussner/MK3_3131_fix_Z…
044a9a4 
…_crash

Fix Z crash and update to 3.13.1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
PR: Configurations PR: General Cleanup PR: Improvement
Projects
None yet
Milestone
1.1.0
Development

Successfully merging this pull request may close these issues.
2 participants
@thinkyhead @jbrazio