Skip to content

Implement per-object top,bottom,initial flow ratios#9809

Draft
logikal wants to merge 1 commit intobambulab:masterfrom
logikal:per-object-top-bottom-flow-ratio
Draft

Implement per-object top,bottom,initial flow ratios#9809
logikal wants to merge 1 commit intobambulab:masterfrom
logikal:per-object-top-bottom-flow-ratio

Conversation

@logikal
Copy link
Copy Markdown
Contributor

@logikal logikal commented Feb 21, 2026

Building locally on osx breaks with every release, it feels like, so pushing to GH just to get the action to build for me.

@zhimin-zeng-bambulab zhimin-zeng-bambulab self-requested a review March 16, 2026 13:26
@tangketan tangketan marked this pull request as ready for review March 16, 2026 13:27
Copy link
Copy Markdown
Collaborator

@tangketan tangketan left a comment

Choose a reason for hiding this comment

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

  1. There is no role called "bottom solid infill"; it's a bit strange to add the bottom_solid_infill_flow_ratio parameter. Please provide a use-case where the new parameter is necessary.
  2. For the first layer of an object, the flow rate is multiplied twice (initial_layer_flow_ratio and bottom_solid_infill_flow_ratio), which is confusing for many people. Is it better to only apply one of the parameters?

@lanewei120 lanewei120 requested a review from tangketan March 16, 2026 13:35
@logikal logikal marked this pull request as draft March 16, 2026 15:52
@logikal
Copy link
Copy Markdown
Contributor Author

logikal commented Mar 16, 2026

Apologies, this was meant to still be a draft, and I'm still experimenting.

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.

4 participants