Skip to content

CURA-12446 top bottom wall count v3 - #21634

Open
Erwan MATHIEU (wawanbreton) wants to merge 7 commits into
mainfrom
CURA-12446_top_bottom_wall_count_v3
Open

CURA-12446 top bottom wall count v3#21634
Erwan MATHIEU (wawanbreton) wants to merge 7 commits into
mainfrom
CURA-12446_top_bottom_wall_count_v3

Conversation

@wawanbreton

@wawanbreton Erwan MATHIEU (wawanbreton) commented May 29, 2026

Copy link
Copy Markdown
Contributor

This PR changes a few existing settings and adds a bunch of new settings, regarding the walls counts:

  • No more "Extra wall count" for skin and infill, but a full count instead. This allows having actually less walls for thoses areas
  • Add specific wall counts for flooring, roofing and initial layer

CURA-12446
Requires Ultimaker/CuraEngine#2334
Supersedes #20565

@github-actions

github-actions Bot commented May 29, 2026

Copy link
Copy Markdown
Contributor

Test Results

24 950 tests   24 888 ✅  53s ⏱️
     1 suites       2 💤
     1 files        60 ❌

For more details on these failures, see this check.

Results for commit 7a67728.

♻️ This comment has been updated with latest results.

@wawanbreton

Copy link
Copy Markdown
Contributor Author

Unit tests are genuinely failing, have to fix some profiles

@github-actions github-actions Bot left a comment

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.

⚠️ Clang-Tidy found issue(s) with the introduced code (1/4)

Comment thread resources/variants/dagoma/dagoma_pro_430_bowden_brass_0.4.inst.cfg
Comment thread resources/variants/dagoma/dagoma_pro_430_bowden_steel_0.4.inst.cfg
Comment thread resources/definitions/hms434.def.json
Comment thread resources/definitions/hms434.def.json
Comment thread resources/definitions/hms434.def.json
Comment thread resources/definitions/hms434.def.json
Comment thread resources/definitions/maker_made_300x.def.json
Comment thread resources/definitions/maker_made_300x.def.json
Comment thread resources/definitions/maker_made_300x.def.json
Comment thread resources/definitions/maker_made_300x.def.json

@github-actions github-actions Bot left a comment

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.

⚠️ Clang-Tidy found issue(s) with the introduced code (2/4)

Comment thread resources/definitions/maker_made_300x.def.json
Comment thread resources/definitions/maker_made_300x.def.json
Comment thread resources/definitions/maker_made_300x.def.json
Comment thread resources/definitions/maker_made_300x.def.json
Comment thread resources/definitions/maker_made_300x.def.json
Comment thread resources/definitions/maker_made_300x.def.json
Comment thread resources/definitions/maker_made_300x.def.json
Comment thread resources/definitions/peopoly_moai.def.json
Comment thread resources/definitions/ultimaker_factor4_plus.def.json
Comment thread resources/definitions/ultimaker_factor4_plus.def.json

@github-actions github-actions Bot left a comment

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.

⚠️ Clang-Tidy found issue(s) with the introduced code (3/4)

Comment thread resources/definitions/ultimaker_method_base.def.json
Comment thread resources/definitions/ultimaker_method_base.def.json
Comment thread resources/definitions/ultimaker_method_base.def.json
Comment thread resources/definitions/ultimaker_s8.def.json
Comment thread resources/definitions/ultimaker_s8.def.json
Comment thread resources/definitions/ultimaker_s8.def.json
Comment thread resources/definitions/ultimaker_s8.def.json
Comment thread resources/definitions/ultimaker_s8.def.json
Comment thread resources/definitions/ultimaker_sketch_sprint.def.json
Comment thread resources/definitions/ultimaker_sketch_sprint.def.json

@github-actions github-actions Bot left a comment

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.

⚠️ Clang-Tidy found issue(s) with the introduced code (4/4)

Comment thread resources/definitions/ultimaker_sketch_sprint.def.json
Comment thread resources/definitions/ultimaker_sketch_sprint.def.json
Comment thread resources/definitions/ultimaker_sketch_sprint.def.json

@rburema Remco Burema (rburema) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we do the upgrader part as a separate bit, or does it belong with this ticket?

I'm fine with either, but if this is meant to be in 5.14, we should at least make a ticket.

Comment thread resources/definitions/fdmprinter.def.json
Comment thread resources/definitions/fdmprinter.def.json
@wawanbreton

Copy link
Copy Markdown
Contributor Author

Do we do the upgrader part as a separate bit, or does it belong with this ticket?

I'm fine with either, but if this is meant to be in 5.14, we should at least make a ticket.

Usually we do it separately, so I created the upgrade script ticket for 5.14 and linked this ticket

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