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

Gyroid infill set as bridge feature. #2

Closed
rongith opened this issue Jul 7, 2018 · 5 comments
Closed

Gyroid infill set as bridge feature. #2

rongith opened this issue Jul 7, 2018 · 5 comments
Labels
bug Something isn't working as intended fixed for the next version That means that you should be able to test it in the latest nightly build

Comments

@rongith
Copy link

rongith commented Jul 7, 2018

Version

version_1.40.1-rc-33-g9cb144dd

Operating system type + version

Windows 10 x64 1803

Behavior

Gyroid infill set as bridge feature.
image

STL/Config (.ZIP) where problem occurs

04513-r01f - I3D-1.zip

@supermerill
Copy link
Owner

I don't understand.

@rongith
Copy link
Author

rongith commented Jul 8, 2018

With Prusa 1.40.1, the gyroid infill is "Internal infill" and shown as dark red color (see joined image). With version_1.40.1-rc-33-g9cb144dd, the gyroid infill is all "Bridge infill" and shown as light blue color.
image
04513-r01f - I3D-1-20180708-191647-(P)04513-Light-MK3-(F)ColorFabb XT - chaud2-(P)PMK3-R0.8-M860-1.40.1+win64.zip.amf.zip

@supermerill
Copy link
Owner

reproduced, i will look at it

@supermerill supermerill added bug Something isn't working as intended fixed for the next version That means that you should be able to test it in the latest nightly build labels Jul 11, 2018
@rongith
Copy link
Author

rongith commented Jul 16, 2018

Didn't know it was intentional to set gyroid infill as bridge. Indeed that way has its logic too although at the moment I prefer not having bridge infill for gyroid as the flow rate is usually not the same at all.

    // require bridge flow since most of this pattern hangs in air
    -virtual bool use_bridge_flow() const { return true; }
    +virtual bool use_bridge_flow() const { return false; }

@supermerill
Copy link
Owner

I think personally that it's unnecessary (infill with bridge setting), as we can tune the infill speed anyway, an it's printing very (enough) well at high speed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working as intended fixed for the next version That means that you should be able to test it in the latest nightly build
Projects
None yet
Development

No branches or pull requests

2 participants