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

Fix No Integer restore in G61 #1

Open
wants to merge 151 commits into
base: bugfix-2.1.x
Choose a base branch
from
Open

Fix No Integer restore in G61 #1

wants to merge 151 commits into from

Conversation

karcaw
Copy link
Owner

@karcaw karcaw commented May 31, 2024

This Fixes the case where no Integers were specified on the G61 command:

G60
G0 X100 Y100 Z20
G61 X Y  

does not return to the last location as :

G60
G0 X100 Y100 Z20
G61 X0 Y0

does. this matches what the spec says:
https://marlinfw.org/docs/gcode/G061.html
which does not even document you can put an integer after the axis letters.

Description

Fix for MarlinFirmware#26222

Requirements

No

Benefits

G61 will work according to the spec.

Configurations

Related Issues

(MarlinFirmware#26222)

karcaw and others added 30 commits May 31, 2024 10:31
This Fixes the case where no Integers were specified on the G61 command:
```
G60
G0 X100 Y100 Z20
G61 X Y  
```
does not return to the last location as :
```
G60
G0 X100 Y100 Z20
G61 X0 Y0
```
does.  this matches what the spec says:
https://marlinfw.org/docs/gcode/G061.html
which does not even document you can put an integer after the axis letters.
The vyper board need a file name which begin with 'main_board_'
Co-authored-by: Scott Lahteine <thinkyhead@users.noreply.github.com>
Co-authored-by: Scott Lahteine <thinkyhead@users.noreply.github.com>
@thinkyhead thinkyhead force-pushed the karcaw-fix-g61 branch 2 times, most recently from 9568ce5 to 85a84cc Compare July 22, 2024 02:43
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.