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

UI move buttons may not restore absolute coordinates. #1682

Closed
fjpanag opened this issue Dec 12, 2023 · 1 comment
Closed

UI move buttons may not restore absolute coordinates. #1682

fjpanag opened this issue Dec 12, 2023 · 1 comment
Labels
⚡ Type: Bug Something isn't working

Comments

@fjpanag
Copy link

fjpanag commented Dec 12, 2023

Mainsail Version:

v2.8.0

Browser:

Firefox

Device:

Desktop PC

Operating System:

Linux

What happened?

When I press a move button, it send the following sequence of commands to the printer:

G91
G1 X+10 F6000
G90

If however the requested movement is not possible (for example the printhead is near its travel limit), the following is generated:

G91
G1 X-100 F6000
Move out of range: -15.000 220.000 8.899 [0.000]

A G90 is never sent, and spuriously leaves the printer in relative coordinates.
This is definitely confusing, and has the great potential to cause a crash.

What did you expect to happen instead?

A G90 should always be sent, regardless of the result of the movement.

How to reproduce this bug?

Home the printer and set the printhead somewhere close to the travel limit.
Press a movement button that would cause a move outside the travel limit.

For example:

  1. G90
  2. G1 X10
  3. Press the button for a 100mm left movement of the printhead.
  4. Observe the terminal; no G90 is issued.

Additional information:

No response

@fjpanag fjpanag added the ⚡ Type: Bug Something isn't working label Dec 12, 2023
@meteyou
Copy link
Member

meteyou commented Dec 13, 2023

It was send, only Klipper doesnt execute it, because the command before exit the current gcode block.

@meteyou meteyou closed this as not planned Won't fix, can't repro, duplicate, stale Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚡ Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants