Skip to content
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

Outer perimeter overhangs are treated as bridge, with "Slow down for curled perimeters" enabled #8018

Open
1 task done
Boerny41 opened this issue Jan 12, 2025 · 5 comments
Labels
enhancement New feature or request

Comments

@Boerny41
Copy link

Is there an existing issue for this feature request?

  • I have searched the existing issues

Is your feature request related to a problem?

Line is detected as "overhang wall"
image

Overhang speed is set to 100mm/s, but the line still uses 50 mm/s as if it is a bridge.
image

As far as I understand, "Slow down for curled perimeters" should prevent the above behavior and force it to use the overhang speed settings. Below is another screenshot with this option disabled, however the line uses the bridge speed in both cases
image

Which printers will be beneficial to this feature?

All

Describe the solution you'd like

Use overhang speed for overhang perimeters when "slow down for curled perimeters" is enabled

Describe alternatives you've considered

Change the line type to bridge. It is very confusing when the line type menu says it is an overhang, but it ignores the overhang settings and instead uses the bridge settings. Even more so when the option that should force overhang walls to use overhang speeds does not do so in this specific case.

Additional context

No response

@Boerny41 Boerny41 added the enhancement New feature or request label Jan 12, 2025
@MxBrnr
Copy link

MxBrnr commented Jan 12, 2025

What version of OrcaSlicer are you using?

@Boerny41
Copy link
Author

2.2.0 latest stable

@Noisyfox
Copy link
Collaborator

Noisyfox commented Jan 14, 2025

Could you provide a test 3mf?

According to what I saw in code, it should not use bridge speed for curled walls:

if (m_config.slowdown_for_curled_perimeters){
ConfigOptionFloatsOrPercents dynamic_overhang_speeds(
{(m_config.get_abs_value("overhang_1_4_speed", ref_speed) < 0.5) ?
FloatOrPercent{100, true} :
FloatOrPercent{m_config.get_abs_value("overhang_1_4_speed", ref_speed) * 100 / ref_speed, true},
(m_config.get_abs_value("overhang_2_4_speed", ref_speed) < 0.5) ?
FloatOrPercent{100, true} :
FloatOrPercent{m_config.get_abs_value("overhang_2_4_speed", ref_speed) * 100 / ref_speed, true},
(m_config.get_abs_value("overhang_3_4_speed", ref_speed) < 0.5) ?
FloatOrPercent{100, true} :
FloatOrPercent{m_config.get_abs_value("overhang_3_4_speed", ref_speed) * 100 / ref_speed, true},
(m_config.get_abs_value("overhang_4_4_speed", ref_speed) < 0.5) ?
FloatOrPercent{100, true} :
FloatOrPercent{m_config.get_abs_value("overhang_4_4_speed", ref_speed) * 100 / ref_speed, true},
(m_config.get_abs_value("overhang_4_4_speed", ref_speed) < 0.5) ?
FloatOrPercent{100, true} :
FloatOrPercent{m_config.get_abs_value("overhang_4_4_speed", ref_speed) * 100 / ref_speed, true},
(m_config.get_abs_value("overhang_4_4_speed", ref_speed) < 0.5) ?
FloatOrPercent{100, true} :
FloatOrPercent{m_config.get_abs_value("overhang_4_4_speed", ref_speed) * 100 / ref_speed, true}});
new_points = m_extrusion_quality_estimator.estimate_extrusion_quality(path, overhang_overlap_levels, dynamic_overhang_speeds,
ref_speed, speed, m_config.slowdown_for_curled_perimeters);
}else{

@Noisyfox
Copy link
Collaborator

I guess it has something to do with this line

} else if (path.role() == erOverhangPerimeter || path.role() == erSupportTransition || path.role() == erBridgeInfill) {
speed = m_config.get_abs_value("bridge_speed");

@Boerny41
Copy link
Author

I loaded the model back in to share a 3mf file and noticed that it does not always use the bridge speed, but only if it is slower than the overhang speed.

GitHub does not allow 3mf files, so I zipped it. If you change the bridge speed to something greater than 100 it will start to use the overhang speed.
overhang.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants