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

Travel moves unnecessarily crossing walls #6251

Open
larsch opened this issue Aug 24, 2019 · 14 comments
Open

Travel moves unnecessarily crossing walls #6251

larsch opened this issue Aug 24, 2019 · 14 comments
Labels
Status: Needs Info Needs more information before action can be taken. Status: Stale ⌛ This issue is over a year old. It might be obsolete or just needs a fresh set of eyes Type: Bug The code does not produce the intended behavior.

Comments

@larsch
Copy link

larsch commented Aug 24, 2019

Application version: 4.2.1

Printer
Custom FDM

travel_moves.3mf.zip

Reproduction steps
Set "Combing" to "Not in skin" or "Inside infill".

Actual results
Cura generates travel moves with many jerks, crossing over skin instead of staying inside print when clearly possible.

Expected results
Normal travel moves without jerks

Additional information
Default slicing where "pointy" travel moves are visible everywhere:

travel_moves

Example: After printing outer wall, nozzle coasts a bit forward, then jerks outside print, crosses print to other hole in print, then jerks back to next outer wall being printing, even though the inside is fully connected.

travel_moves_2

image

@larsch larsch added the Type: Bug The code does not produce the intended behavior. label Aug 24, 2019
@smartavionics
Copy link
Contributor

Turn off the avoid printed parts when travelling setting.

@larsch
Copy link
Author

larsch commented Aug 24, 2019

Turning off Avoid Printed Parts does prevent the jerk travel moves. OK for this print.

@Ghostkeeper
Copy link
Collaborator

Ghostkeeper commented Aug 26, 2019

Yeah, looks buggy to me. I can see that it's combing correctly along the first inner wall, but when it needs to separate from that wall it decides to do a moveOutside which crosses the wall, and then makes a straight run for the moveOutside-location of the destination because it sees that it needs to cross the outer wall anyway (because it's now outside). It shouldn't do that moveOutside there.

It's probably doing that because it sees that the destination is already unreachable with combing. After all, combing is not allowed through skin as per the value of the Combing Mode setting.

@smartavionics
Copy link
Contributor

Yes, it's because it can't comb within the skin. Notice that the spikes don't occur on the layers that don't have skin.

@smartavionics
Copy link
Contributor

It would be better if it did a non-combed direct travel move with retraction (and, hopefully, z-hop to avoid wrecking the skin).

@Liger0
Copy link

Liger0 commented Sep 15, 2019

I also have the nozzle going into empty areas going from start point to the ending one on a straight line instead of staying within the model on cura 4.3 beta, no matter if I use non in skin, in infill or all.

@smartavionics
Copy link
Contributor

Please provide the .3mf file.

@Liger0
Copy link

Liger0 commented Sep 15, 2019

Literally any model does it.

@smartavionics
Copy link
Contributor

Unless you provide an example, I cannot help.

@Liger0
Copy link

Liger0 commented Sep 15, 2019

Combing.zip

@smartavionics
Copy link
Contributor

Thanks for the file. I have sliced it but can't see an obvious problem yet, please say what layer is causing the issue and maybe show an image of the problem. Thanks.

@Liger0
Copy link

Liger0 commented Sep 15, 2019

I am not very good understanding everything combing does, but I think the travel on the top are not supposed to be there.
combing

@smartavionics
Copy link
Contributor

Ah, I see. Thing is, I don't get those travels when I slice with my cura. I think I fixed the bug that causes them a long time ago but the cura developers have not yet incorporated the fix. That was Ultimaker/CuraEngine#973.

Screenshot_2019-09-15_20-22-00

@Ghostkeeper Ghostkeeper changed the title Travel move Travel moves unnecessarily crossing walls Dec 23, 2021
@GregValiant GregValiant added Status: Needs Info Needs more information before action can be taken. Status: Stale ⌛ This issue is over a year old. It might be obsolete or just needs a fresh set of eyes labels Nov 22, 2024
@GregValiant
Copy link
Collaborator

I'm cleaning house.
Is this still a problem with current Cura versions (5.8.0 and up)? Can I close this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Info Needs more information before action can be taken. Status: Stale ⌛ This issue is over a year old. It might be obsolete or just needs a fresh set of eyes Type: Bug The code does not produce the intended behavior.
Projects
None yet
Development

No branches or pull requests

6 participants