Skip to content

Fix two related travel move bugs - #513

Open
jeffc wants to merge 2 commits into
GridSpace:rel-nextfrom
jeffc:fix-drag
Open

Fix two related travel move bugs#513
jeffc wants to merge 2 commits into
GridSpace:rel-nextfrom
jeffc:fix-drag

Conversation

@jeffc

@jeffc jeffc commented Aug 30, 2026

Copy link
Copy Markdown

Let me know if you want me to separate this out into two PRs for the two bug fixes:

Bug 1

Update travel moves that are entirely within the part boundary (eg, going from the end of one cutting layer to the beginning of the next) to perform a small (0.1mm) "hop" up, then a rapid in-plane travel, and then finally a rapid move back to the previous z height before starting the plunge move.

Before (large step-down and step-over in order to make the visualization clearer):

image

After:

image

Note that instead of moving at "plunge speed" from the innermost ring of the last layer directly to the outermost ring of the next, we now fast-move up 0.1mm and then over to the plunge point before plunging straight down. (hence the stacked plunge lines on the right side of the screenshot

The code for this is in two different places due to the structure of the nested conditional.

Bug 2

Moves between concentric offsets in an area clear operation were previously getting omitted as fast moves (G0), meaning that we were doing a (relatively short, usually) travel-speed move through uncut material. Notice in the first screenshot how the lines between each concentric circle are blue (travel speed), whereas in the second screenshot they're green (cutting speed). We only force cutting speed if we're making a small move with a zero or insignificant-but-negative Z delta, otherwise we leave it alone.

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.

1 participant