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

Touch Mi: Add magnet Y position #15166

Merged
merged 4 commits into from Sep 4, 2019
Merged

Touch Mi: Add magnet Y position #15166

merged 4 commits into from Sep 4, 2019

Conversation

kakou-fr
Copy link
Contributor

@kakou-fr kakou-fr commented Sep 4, 2019

Requirements

  • Filling out this template is required. Pull Requests without a clear description may be closed at the maintainers' discretion.

also correct a bug in motion.cpp :
do_blocking_move_to(current_position[Y_AXIS], ry, current_position[Z_AXIS], fr_mm_s);
Y_AXIS => X_AXIS

Description

This patch allow to specify an Y position for the touch mi magnet.

Benefits

On Core-XY the magnet used to unlock the pin of a touch mi could be at an specific XY position and not only X. This patch allow to define a X, Y or XY position for the magnet.

@thinkyhead thinkyhead merged commit 6b1c4de into MarlinFirmware:bugfix-2.0.x Sep 4, 2019
@thinkyhead thinkyhead changed the title touch mi add Y position for magnet Touch Mi: Add magnet Y position Sep 4, 2019
@SrGeek
Copy link

SrGeek commented Apr 15, 2020

How do I implement feature in Marlin. My touch mi probe is at the end of the Y axis.

@thinkyhead
Copy link
Member

@SrGeek
This Issue Queue is for Marlin bug reports and development-related issues, and we prefer not to handle user-support questions here. (As noted on this page.) For best results getting help with configuration and troubleshooting, please use the following resources:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants