-
-
Notifications
You must be signed in to change notification settings - Fork 19.2k
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
[BUG] G61 does not return to saved position when Axis specified without an offset #26222
Closed
1 task done
Labels
Fix Included
A fix is included in the description
Comments
yup, still valid |
yup, still valid |
still not fixed |
Any chance we can get this merged? |
Since you've provided a fix, please open a proper Pull Request for review. See Contributing Code with Pull Requests for more details. |
I made one,... Seems I pointed it at the wrong master: karcaw#1 |
Added proper PR |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Did you test the latest
bugfix-2.1.x
code?Yes, and the problem still exists.
Bug Description
When you use the G61 Return to Saved Position feature, and specify any axis to restore, and do not put a offset value, it does not return to the saved location, it does not change that axis at all. It stems from Line 89:
Marlin/Marlin/src/gcode/feature/pause/G61.cpp
Lines 87 to 89 in 11f98ad
Which defaults to the current position, and not to
stored_position[slot][i]
as it should.The online line documentation here: https://marlinfw.org/docs/gcode/G061.html
does not even mention that offsets are an option, so anyone using this command with axis restores is bound to fail.(As I did)
Bug Timeline
In the original code of G61 (#16557)
Expected behavior
I expect it to move back to the proper position.
Actual behavior
It does not move
Steps to Reproduce
for Example if you run:
No movement is seen on the G61. if instead you change that to
G61 X0 Y0
it will properly move back to theX10 Y10
position.a bare
G61
does work properly.Version of Marlin Firmware
Marlin 2.1.2
Printer model
Creality Ender 3 Pro
Electronics
Stock 4.2.2 Board
Add-ons
No response
Bed Leveling
UBL Bilinear mesh
Your Slicer
Prusa Slicer
Host Software
OctoPrint
Don't forget to include
Configuration.h
andConfiguration_adv.h
.Additional information & file uploads
EF8_G16Debug.zip
The text was updated successfully, but these errors were encountered: