Skip to content

[PP-605] Add machine_nozzle_size to variants #20463

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: main
Choose a base branch
from

Conversation

Frederic98
Copy link
Contributor

Description

This PR adds the machine_nozzle_size to the S-line variants that did not have them yet. This makes our jinja templates a lot cleaner and easier.

This does not change any setting trees, as fdmprinter already defines the nozzle size as 0.4 as default. However, the nozzle size is strictly a property of the variant, so it's good to explicitly set it there.

The CC 0.4 variants already defined the machine_nozzle_size as 0.4, so only the AA and BB cores are affected.

Copy link
Contributor

github-actions bot commented Mar 31, 2025

Test Results

23 607 tests  ±0   23 605 ✅ ±0   47s ⏱️ -1s
     1 suites ±0        2 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 1045dc9. ± Comparison against base commit 0fd7766.

♻️ This comment has been updated with latest results.

@pkuiper-ultimaker pkuiper-ultimaker changed the base branch from main to 5.10 April 4, 2025 10:13
@pkuiper-ultimaker pkuiper-ultimaker changed the base branch from 5.10 to main April 4, 2025 10:13
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