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

support roof is suspended! #11

Closed
zhangjiangzhao123 opened this issue Mar 24, 2023 · 6 comments
Closed

support roof is suspended! #11

zhangjiangzhao123 opened this issue Mar 24, 2023 · 6 comments

Comments

@zhangjiangzhao123
Copy link

Dear ThomasRahm, I may have discovered a glitch. When I set ”support_roof_pattern“=Lines, and Support Roof Line Directions=[0,90], it will produce the effect of the picture below. Obviously part of the roof is suspended. This model will not print successfully.
image

image

support_roof_pattern“=Zigzag is also have this problem

image

Support Roof Line Directions=`[90,180] either.
image

My test model and gcode files are in the following zip file.
test_model.zip

@ThomasRahm
Copy link
Owner

The first two pictures seem fine to me. Only the bottom-most interface layer is supported as this layer will support the interface above.
The last two do not. This line should be supported

@ThomasRahm
Copy link
Owner

ThomasRahm commented Mar 29, 2023

The issue with the last two is fixed (in the code). Thank you for reporting it.
Seems to be that the behavior of regular for lines is a bit different to other infill patterns, so the first two may also be considered a bug. Not sure if this one is fixable, as the code assumes that each roof area the roof area above it supports. As a workaround i recommend setting Support Roof Wall Line Count (new setting in 5.3) to 1.

@zhangjiangzhao123
Copy link
Author

Thanks, The last two situations have been resolved well. For the first two, adding an outer contour(wall) is indeed a better option. I will try to follow your advice.

@zhangjiangzhao123
Copy link
Author

By the way, I know if Infill class: fill_gaps = false , no dangling segments like last two will be generated. It seems that these line segments are extra generated. So where might it come from?

@ThomasRahm
Copy link
Owner

I don't know. To determine how to support the support roof I just call the Infill class with the same parameters that are later used in FFFGcodeWriter when calculating the support roof. My code does not handle the placement of the roof lines.
The only reasons I did not see the change of the Infill class that caused the issue earlier are that the Infill class has a lot of optional parameters and using Polygons* for a bool does not cause an error.

@zhangjiangzhao123
Copy link
Author

Thank you. This may be an upstream bug.

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

No branches or pull requests

2 participants