Skip to content

PP-740 Improve bridging for F4plus - #21775

Open
THeijmans wants to merge 1 commit into
mainfrom
PP-740-Improve-bridging-F4plus
Open

PP-740 Improve bridging for F4plus#21775
THeijmans wants to merge 1 commit into
mainfrom
PP-740-Improve-bridging-F4plus

Conversation

@THeijmans

@THeijmans THeijmans commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Description

As discussed in eCCB

Type of change

  • Printer definition file(s)

How Has This Been Tested?

  • Loaded in Cura local build
  • Printed with several materials

Test Configuration:

  • Operating System:

Checklist:

@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/2)

"bridge_interlace_lines": { "value": "True" },
"bridge_skin_density": { "value": "70" },
"bridge_skin_density": { "value": "100" },
"bridge_skin_material_flow":

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.

diagnostic-circular-dependency
Circular dependency detected across settings: bridge_skin_material_flow -> bridge_skin_material_flow. Check the value formulas for these settings in this file and its parent definitions.

"acceleration_wall_x": { "value": "acceleration_print" },
"acceleration_wall_x_roofing": { "value": "acceleration_wall" },
"bottom_skin_expand_distance": { "value": "expand_skins_expand_distance / 2" },
"bridge_enable_more_layers": { "value": "False" },

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.

⚠️ diagnostic-definition-experimental-setting ⚠️
Setting bridge_enable_more_layers is still experimental and should not be used in default profiles

"acceleration_wall_x_roofing": { "value": "acceleration_wall" },
"bottom_skin_expand_distance": { "value": "expand_skins_expand_distance / 2" },
"bridge_enable_more_layers": { "value": "False" },
"bridge_interlace_lines": { "value": "True" },

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.

⚠️ diagnostic-definition-experimental-setting ⚠️
Setting bridge_interlace_lines is still experimental and should not be used in default profiles

"bridge_enable_more_layers": { "value": "False" },
"bridge_interlace_lines": { "value": "True" },
"bridge_skin_density": { "value": "70" },
"bridge_skin_density": { "value": "100" },

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.

⚠️ diagnostic-definition-experimental-setting ⚠️
Setting bridge_skin_density is still experimental and should not be used in default profiles

"bridge_interlace_lines": { "value": "True" },
"bridge_skin_density": { "value": "70" },
"bridge_skin_density": { "value": "100" },
"bridge_skin_material_flow":

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.

⚠️ diagnostic-definition-experimental-setting ⚠️
Setting bridge_skin_material_flow is still experimental and should not be used in default profiles

"value": "(bridge_wall_material_flow + material_flow) / 2"
"value": "bridge_skin_material_flow"
},
"bridge_skin_material_flow_2": { "maximum_value": "250" },

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.

⚠️ diagnostic-definition-experimental-setting ⚠️
Setting bridge_skin_material_flow_2 is still experimental and should not be used in default profiles

"value": "bridge_skin_material_flow"
},
"bridge_skin_material_flow_2": { "maximum_value": "250" },
"bridge_skin_material_flow_3": { "maximum_value": "250" },

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.

⚠️ diagnostic-definition-experimental-setting ⚠️
Setting bridge_skin_material_flow_3 is still experimental and should not be used in default profiles

@@ -75,16 +76,16 @@
"bridge_wall_material_flow":

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.

⚠️ diagnostic-definition-experimental-setting ⚠️
Setting bridge_wall_material_flow is still experimental and should not be used in default profiles

"value": "150"
"value": "material_flow"
},
"bridge_wall_min_length": { "value": "line_width*2" },

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.

⚠️ diagnostic-definition-experimental-setting ⚠️
Setting bridge_wall_min_length is still experimental and should not be used in default profiles

"bridge_wall_speed": { "value": "speed_wall/2" },
"bridge_wall_speed": { "value": 10 },
"cool_min_layer_time": { "value": 6.5 },
"cool_min_layer_time_overhang": { "value": 12 },

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.

⚠️ diagnostic-definition-experimental-setting ⚠️
Setting cool_min_layer_time_overhang is still experimental and should not be used in default 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 (2/2)

"bridge_wall_speed": { "value": 10 },
"cool_min_layer_time": { "value": 6.5 },
"cool_min_layer_time_overhang": { "value": 12 },
"cool_min_layer_time_overhang_min_segment_length": { "value": 1.5 },

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.

⚠️ diagnostic-definition-experimental-setting ⚠️
Setting cool_min_layer_time_overhang_min_segment_length is still experimental and should not be used in default profiles

"skin_support_material_flow": { "value": "material_flow" },
"skin_support_speed": { "value": "speed_wall_0" },
"skirt_brim_speed": { "maximum_value": 125 },
"small_feature_speed_factor": { "value": 100 },

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.

⚠️ diagnostic-definition-experimental-setting ⚠️
Setting small_feature_speed_factor is still experimental and should not be used in default profiles

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