Skip to content

Adjust combined infill flow to correct underextrusion #14620

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xiphmont
Copy link

@xiphmont xiphmont commented Jun 4, 2025

Resubmitted without extraneous whitespace changes.

Combined infill currently sets extrusion flow as if implicit extrusion cross-section scales proportionally with layer height. That would be true with a rectangular or cross-sectional extrusion model, but with a rounded rectangle model, the proportion of the extrusion taken by the two semicircles increases and the center rectangle narrows.

This profile change with increased height/thickness causes underextrusion when width and spacing are both kept constant. Delivered density falls below the requested value and, in the worst case, the underextrusion can trigger "extrusion width too small?" errors as the rounded rectangle inverts past a circle.

This change:

1) rescales rounded rectangle width to preserve requested fill density
when fill surface thickness != layer height.

2) relaxes the requirement for aligned spacing in the event requested
density is 100% (treating the fill as solid for Flow calculation, though
preserving it as InternalInfill).

Combined infill currently sets extrusion flow as if implicit extrusion
cross-section scales proportionally with layer height.  That would be
true with a rectangular or cross-sectional extrusion model, but with a
rounded rectangle model, the proportion of the extrusion taken by the
two semicircles increases and the center rectangle narrows.

This profile change with increased height/thickness causes
underextrusion when width and spacing are both kept
constant. Delivered density falls below the requested value and, in
the worst case, the underextrusion can trigger "extrusion width too
small?" errors as the rounded rectangle inverts past a circle.

This change:

    1) rescales rounded rectangle width to preserve requested fill density
    when fill surface thickness != layer height.

    2) relaxes the requirement for aligned spacing in the event requested
    density is 100% (treating the fill as solid for Flow calculation, though
    preserving it as InternalInfill).
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

Successfully merging this pull request may close these issues.

1 participant