Skip to content

[CURA-13265] Fix infill (wall) overlap for infill-support. - #2362

Merged
HellAholic merged 3 commits into
mainfrom
CURA-13265_fix_infill_support_overlap
Aug 20, 2026
Merged

[CURA-13265] Fix infill (wall) overlap for infill-support.#2362
HellAholic merged 3 commits into
mainfrom
CURA-13265_fix_infill_support_overlap

Conversation

@rburema

Copy link
Copy Markdown
Member

Wall-width is already taken into account (but maybe not at the time of writing?), so when subtracting half a wall-width over that, it made it seem like the infill overlap setting wasn't working properly.

Wall-width is already taken into account (but maybe not at the time of writing?), so when subtracting half a wall-width over that, it made it seem like the infill overlap setting wasn't working properly.

CURA-13265
@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Test Results

31 tests   31 ✅  4s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit 106f614.

♻️ This comment has been updated with latest results.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, that should improve adhesion between the skin support and the outer walls.
Sorry for the mistake 😅
When thinking about it though, I probably did that to avoid the protrusion of the infill into the outer walls. HellAholic could you make a test print as part of QA to check the quality ?

@rburema

Copy link
Copy Markdown
Member Author

Erwan MATHIEU (@wawanbreton) Don't worry about it!

Also, if the quality is a problem, we should change that with parameters I think. Worst case, we can split it out to a new one.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'C++ Benchmark'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.50.

Benchmark suite Current: 9a6c743 Previous: fbf37fc Ratio
InfillTest/Infill_generate_connect/0/400 5.804878534799016 ms/iter 3.648410709524255 ms/iter 1.59
HolesWallTestFixture/InsetOrderOptimizer_getInsetOrder/3 0.0002625348999732111 ms/iter 0.00017382738462409593 ms/iter 1.51
HolesWallTestFixture/InsetOrderOptimizer_getInsetOrder/9999 0.011970930861527649 ms/iter 0.007572545422304867 ms/iter 1.58
SimplifyTestFixture/simplify_slot_noplugin 3.789785515326669 ns/iter 2.1604040436005425 ns/iter 1.75

This comment was automatically generated by workflow using github-action-benchmark.

CC: Jelle Spijker (@jellespijker) Erwan MATHIEU (@wawanbreton) Casper Lamboo (@casperlamboo) HellAholic

@HellAholic

Copy link
Copy Markdown
Contributor

Nice, that should improve adhesion between the skin support and the outer walls. Sorry for the mistake 😅 When thinking about it though, I probably did that to avoid the protrusion of the infill into the outer walls. HellAholic could you make a test print as part of QA to check the quality ?

Before:
8689

After:
8690

Using lightning infill and 0.04mm infill overlap (PLA-0.2mm)

@HellAholic
HellAholic merged commit 07475e3 into main Aug 20, 2026
35 checks passed
@HellAholic
HellAholic deleted the CURA-13265_fix_infill_support_overlap branch August 20, 2026 11:02
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.

3 participants