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

Ensure Vertical Shell Thickness interferes with solid top layer count #60

Closed
Endarer opened this issue Dec 11, 2016 · 3 comments
Closed
Labels

Comments

@Endarer
Copy link

Endarer commented Dec 11, 2016

Version

1.31.6-prusa3d

Operating system type + version

OSX

Behavior

If ensure vertical shell thickness is enabled and in use for a given feature. Infill will resume after enhanced vertical wall is complete; reducing the solid top layer count.

Problem can be reproduced using the default prusa profiles on the model listed below.
http://www.thingiverse.com/thing:1794872
In this case the solid top layer count is reduced to 1 for the middle features

First solid top layer
vertical wall bug

Second to last layer. (First layer of infill after enhanced vertical wall completion)
vertical wall bug2

First solid top Layer with ensure vertical shell thickness disabled
vertical wall bug3

@bubnikv bubnikv added the bug label Dec 15, 2016
@m0n5t3r
Copy link

m0n5t3r commented Dec 22, 2016

just got bitten by this; it kills all solid infill in some places, not just top layer, see attachment

saintflint_mk8.zip

@bubnikv
Copy link
Collaborator

bubnikv commented Dec 22, 2016

Thanks for feedback. I will look into this issue for the next release.

bubnikv added a commit that referenced this issue Jan 2, 2017
Number of solid top/bottom layers ignored,
Ensure Vertical Shell Thickness interferes with solid top layer count
#79
#60
@bubnikv
Copy link
Collaborator

bubnikv commented Jan 2, 2017

Fixed by 17db5bf

@bubnikv bubnikv closed this as completed Jan 2, 2017
domesticatedviking pushed a commit to domesticatedviking/PrusaSlicer that referenced this issue Sep 27, 2019
 - replace cast to extrusionPath by oop methods.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants