Skip to content

Conversation

KaarelP2rtel
Copy link
Contributor

@KaarelP2rtel KaarelP2rtel commented Feb 6, 2025

Description

Currently when starting the spiral vase the extrusion rate is ramped from 0 to 100% on the first layer and from 100% to 0% on the last layer. In some cases it can lead to underextrusion at the beginning and end of the spiral.

This change adds minimum flow rate options for the beginning and the end of the vase. This means that instead of ramping from 0% to 100% it instead ramps from for example 20% to 100%.

This issue has been reported in SuperSlicer
supermerill/SuperSlicer#4195

Screenshots/Recordings/Graphs

screenshot_2025-02-06-150503_grim
screenshot_2025-02-06-150430_grim
screenshot_2025-02-06-150450_grim

Tests

Created a Spiral Vase slice that is limited by flow rate. Observed Flow Color scheme of the slice to verify that flow rate is now scaled from and to the selected percentage.

@KaarelP2rtel
Copy link
Contributor Author

Original PR to PrusaSlicer:
prusa3d/PrusaSlicer#13329

@KaarelP2rtel KaarelP2rtel force-pushed the minimum_spiral_transition_flowrates branch from 46a7ed2 to 73469cd Compare February 6, 2025 13:11
@KaarelP2rtel KaarelP2rtel marked this pull request as ready for review February 6, 2025 13:11
Noisyfox
Noisyfox previously approved these changes Feb 8, 2025
Copy link
Collaborator

@Noisyfox Noisyfox left a comment

Choose a reason for hiding this comment

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

LGTM overall. Thanks

Currently when starting the spiral vase the extrusion rate is ramped
from 0 to 100% on the first layer and from 100% to 0% on the last layer.
In some cases it can lead to underextrusion at the beginning and end of
the spiral.

This change adds minimum flow ratio options for the beginning and the end
of the vase. This means that instead of ramping from 0% to 100% it
instead ramps from for example 20% to 100%.

This issue has been reported in SuperSlicer
supermerill/SuperSlicer#4195
@KaarelP2rtel
Copy link
Contributor Author

@Noisyfox Done. Though IMO all of the other ratios should be percentages instead however that will obviously not be a simple change.
I also went ahead and renamed "rate" to "ratio" as that is how all the other ratios are named.

Copy link
Collaborator

@Noisyfox Noisyfox left a comment

Choose a reason for hiding this comment

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

Awesome, thanks

@Noisyfox
Copy link
Collaborator

Though IMO all of the other ratios should be percentages instead however that will obviously not be a simple change.

Yeah agreed.

@Noisyfox Noisyfox merged commit 830c1ac into SoftFever:main Feb 17, 2025
16 checks passed
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