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

Cura 9790 reconfigure gradual infill #1771

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

rijkvanmanen
Copy link
Contributor

@rijkvanmanen rijkvanmanen commented Nov 15, 2022

Currently the infill density defines the top density when enabling gradual infill. With this PR the infill density is defining the bottom density when enabling gradual infill. This causes gradual infill to only add a few layers of extra infill, without modifying the main infill density.

frontend PR: Ultimaker/Cura#13827

CURA-9790

Only apply gradual infill when parent setting is true. CURA-9790
instead of reducing the infill density by gradual infill towards the bottom of the print, the infill density is now increased towards the top of the print. CURA-9790
Gradual infill failed for steps > 1 due to type error. CURA-9790
Gradual infill in support is handled in a different place. Added the changes to support as well. CURA-9790
@github-actions
Copy link
Contributor

github-actions bot commented Nov 15, 2022

Unit Test Results

25 tests  ±0   25 ✔️ ±0   11s ⏱️ -1s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ±0 

Results for commit c84b741. ± Comparison against base commit 9e4212e.

♻️ This comment has been updated with latest results.

@rijkvanmanen rijkvanmanen marked this pull request as ready for review January 16, 2023 14:03
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.

2 participants