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

Occasionally Travel Moves are Generated With Extrusions. #4554

Open
ruevs opened this issue Oct 1, 2018 · 11 comments
Open

Occasionally Travel Moves are Generated With Extrusions. #4554

ruevs opened this issue Oct 1, 2018 · 11 comments
Labels
Thin walls Verified bug This issue has been reproduced by a developer

Comments

@ruevs
Copy link

ruevs commented Oct 1, 2018

Version

Version 1.3.1-dev
Build commit: d17659b
32bit
Slic3r-master.2018.09.08.1848.d17659b.32bit.zip

Operating system type + version

Windows XP SP3 32bit

Wrong Behavior

Occasionally moves that should be travel moves are generated with extrusions.
Models similar to the attached one with "three spokes" occasionally show this on layers other than the first one as well.
slic3rbug

STL/Config (.ZIP) where problem occurs

Slic3rBug.zip
config.zip

Expected behavior

This is a regression of sorts. For example Slic3r-master.2018.04.26.1669.8a93fbb.32bit.zip (which I had laying around) does NOT have this problem when slicing the exact same object running on the same machine (so the exact same configuration as above).

Slic3r-master.2018.04.26.1669.8a93fbb.32bit.gcode.zip

slic3r-master 2018 04 26 1669 8a93fbb 32bitfirstlayer
slic3r-master 2018 04 26 1669 8a93fbb 32bit

Initially I posted this here #3755 but I suspect it is not the same issue so I am opening this one.
@lordofhyphens
Copy link
Member

So the problem goes away when you run the STL through netfabb. Joy.

Slic3rBug_fixed.zip

@ruevs
Copy link
Author

ruevs commented Oct 17, 2018

Thank you for looking at this.

The file you uploaded slices fine indeed. However the mesh is very different from the original. How did you "run it through netfabb"?

At the same time the original is a perfectly valid STL file according to Netfabb itself:
slic3rbug_innetfabb
The part repair tool reports that everything is fine and does not change the geometry at all when executed - at least in Netfabb version 7.4.0 532 that I have.

So what does this mean for the Slic3r behavior? What about the fact that an older build slices it fine?

@lordofhyphens
Copy link
Member

Service.netfabb.com is what I meant.

I have not done a bisect yet ( it is a very long process to do so)

@platsch
Copy link
Member

platsch commented Nov 20, 2018

I can confirm the bug, having the same issue with several models. The extrusions are actually printed and not only generated for the preview. Blaming the stl files to be inconsistent is a an easy, but not very helpful answer. I'm having this issue with a number of different files which are sliced correctly by older versions of Slic3r. Runnung every second model through netfabb is slightly annoying...

Unfortunately I have no clue of what could be the reason, didn't do substantial debugging though.

@VanessaE
Copy link
Collaborator

VanessaE commented Dec 2, 2018

I am experiencing this artifact, too (commit 1e4fb2d).

image

(there is not supposed to be any geometry at the bottom of that open space)

Stl: turn signal stalk repair.stl.zip
Config: config-20181201.ini.zip

@lordofhyphens
Copy link
Member

reproduction_case.zip
Here's a cut-down version of @VanessaE model.

Looks like it's in the thinwall code.

@lordofhyphens lordofhyphens added Verified bug This issue has been reproduced by a developer Thin walls labels Dec 2, 2018
@lordofhyphens
Copy link
Member

The pseudocode for the regression test looks like thus:

Given: The 3D model [reproduction_case.zip] (https://github.com/slic3r/Slic3r/files/2636486/reproduction_case.zip) and a default config,
When: the model is plated centered at 0,0 and thin_walls=1

Then: No extrusion moves are generated from X1.856 Y-6.933 to X1.793 Y-6.899

@supermerill
Copy link
Collaborator

supermerill commented Dec 2, 2018

My previous thin wall pr is seriously flawed. You should revert it or i have to finish the new one.

@ruevs
Copy link
Author

ruevs commented Apr 18, 2019

Any progress on this?
Slic3r-master.2019.04.13.2318.85dd6633.32bit-PR.zip
Still has exactly the same problem.

@supermerill
Copy link
Collaborator

supermerill commented Apr 18, 2019

should be resolved on the branch merill_thinwall

should be downloadable in the dev branches like here
https://dl.slic3r.org/dev/
search for PR4790

@ruevs
Copy link
Author

ruevs commented Mar 19, 2020

It seem to be fixed in this AppVeyor build artifact:
https://ci.appveyor.com/project/lordofhyphens/slic3r/builds/31012289
from here:
#4790

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Thin walls Verified bug This issue has been reproduced by a developer
Projects
None yet
Development

No branches or pull requests

5 participants