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

Probe wizard move to #20167

Merged
merged 7 commits into from
Nov 18, 2020
Merged

Probe wizard move to #20167

merged 7 commits into from
Nov 18, 2020

Conversation

wmariz
Copy link
Contributor

@wmariz wmariz commented Nov 17, 2020

Description

Add setting to move to specific position on PROBE_OFFSET_WIZARD

Benefits

Add setting to move to specific position on PROBE_OFFSET_WIZARD

Configurations

Configuration_adv.zip

Related Issues

#19650

@qwewer0
Copy link
Contributor

qwewer0 commented Nov 17, 2020

This doesn't seem to work for me. Ender 3, BLTouch, SKR Mini E3 v1.2

I don't think a G0/G1 call is the right way to move the nozzle to the desired position. While I wasn't able to make do_blocking_move_to_xy to work in menu_probe_offset.cpp (always got stepper motor siren), I still think that or some other function should be called instead of a G0/G1 move.

If it would needs to be a G0/G1 call, then I think it should look something like this instead:

  #if ENABLED(Z_SAFE_HOMING)
    queue.enqueue_one_now(PSTR("G0 X" STRINGIFY(Z_SAFE_HOMING_X_POINT) " Y" STRINGIFY(Z_SAFE_HOMING_Y_POINT) " F" STRINGIFY(HOMING_FEEDRATE_XY)));
  #else
    queue.enqueue_one_now(PSTR("G0 X" STRINGIFY(X_CENTER) " Y" STRINGIFY(Y_CENTER) " F" STRINGIFY(HOMING_FEEDRATE_XY)));
  #endif

It does not make sense to me to have PROBE_OFFSET_MOVE_TO_X/Y, because if someone has the Z_SAFE_HOMING set to a different position then PROBE_OFFSET_MOVE_TO_X/Y would need to be set the same to make the measurement accurate, and so PROBE_OFFSET_MOVE_TO_X/Y would be obsolete. So instead, we could just use the Z_SAFE_HOMING position if it is enabled and use X/Y_CENTER otherwise.

@thisiskeithb thisiskeithb added F: Calibration Needs: Testing Testing is needed for this change labels Nov 17, 2020
@thinkyhead
Copy link
Member

I've modified the code to follow the methodology of other menu items that want to home or do a move, so it should behave better.

@thinkyhead thinkyhead merged commit 5768ee0 into MarlinFirmware:bugfix-2.0.x Nov 18, 2020
@wmariz wmariz deleted the Probe-Wizard-Move-to branch November 18, 2020 04:13
@qwewer0
Copy link
Contributor

qwewer0 commented Nov 18, 2020

@thinkyhead, @wmariz
When I go to Probe Offset Wizard in the LCD

  1. It shows "Homing XYZ"
  2. Before homing is done the LCD goes back to the status screen
  3. After it homed, it doesn't move to bed center (PROBE_OFFSET_WIZARD_XY_POS), stays at homing position
  4. If I select Probe Offset Wizard again, LCD shows "Moving..."
  5. It starts to move to home direction (X/Y min) until it hits one of the endstops
  6. After it hit an endstop it starts to move to X/Y max and crashes into the frame (at this point I shot down the printer)

Ender 3, SKR Mini E3 v1.2, Latest Bugfix
Configuration.zip

If needed, I could open an issue for this, but I hope that it can be solved with some easy fix.

FhlostonParadise pushed a commit to FhlostonParadise/Marlin that referenced this pull request Nov 21, 2020
Kannix2005 pushed a commit to Kannix2005/Marlin-1 that referenced this pull request Dec 7, 2020
vgadreau pushed a commit to vgadreau/Marlin that referenced this pull request Dec 9, 2020
tharts pushed a commit to tharts/Marlin that referenced this pull request Jan 6, 2021
dpreed pushed a commit to dpreed/Marlin_2.0.x that referenced this pull request Feb 5, 2021
dpreed pushed a commit to dpreed/Marlin_2.0.x that referenced this pull request Feb 5, 2021
kpishere pushed a commit to kpishere/Marlin that referenced this pull request Feb 19, 2021
W4tel-BiDi pushed a commit to W4tel-BiDi/Marlin that referenced this pull request Apr 5, 2021
chrisjenda pushed a commit to chrisjenda/Marlin that referenced this pull request Apr 8, 2021
chrisjenda pushed a commit to chrisjenda/Marlin that referenced this pull request Apr 11, 2021
thinkyhead pushed a commit to thinkyhead/Marlin that referenced this pull request Apr 28, 2021
thinkyhead pushed a commit to thinkyhead/Marlin that referenced this pull request Apr 29, 2021
thinkyhead pushed a commit that referenced this pull request Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
F: Calibration Needs: Testing Testing is needed for this change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants