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

Default Fan speed ignored #3380

Open
adripo opened this issue Oct 28, 2022 · 4 comments
Open

Default Fan speed ignored #3380

adripo opened this issue Oct 28, 2022 · 4 comments

Comments

@adripo
Copy link

adripo commented Oct 28, 2022

What happened?

min_fan_speed is set at 35%, but gcode is created starting from 0%

image

image

Project file & How to reproduce

Any project behave the same.

Filament settings -> Default fan speed: 35%
Gcode Preview -> View Fan speed -> Values below default fan speed

Version

2.4.58.5

Operating system

windows 10

Printer model

No response

@adripo
Copy link
Author

adripo commented Oct 28, 2022

I would expect to have:

  • 1 layer: 0%
  • 2 layer: 35%
  • 3 layer: ~60%
  • 4 layer: 100%

image

Am I missing something?

@diggit
Copy link

diggit commented Oct 28, 2022

"Full fan speed at layer" is definitely confusing. It means, that at layer 4, this limiter will stop limiting and fan will be on its target speed. Thus:

  • layer 1: 0%
  • layer 2: 12%
  • layer 3: 23%
  • layer 4: 35 %
    I am unsure, why 5th layer gets to 80~90 %. Too short layer time I guess?

@adripo
Copy link
Author

adripo commented Oct 30, 2022

What is the purpose of default fan speed then? It doesn't affect the "minimum PWM that my fan needs to work" as explained in the details. I also checked Run the fan at default speed when possible but no change.

@legend069
Copy link

default fan speed is used for extrusion roles that currently don't have a fan sped override set

supermerill added a commit that referenced this issue Nov 14, 2023
 . now easier to add new control and do complicate stuff in the cooling buffer.
 . less overall number of lines of code for better results.
 . add 6 new fan control.
 . now if the disable_fan_first_layers is at 0 and full_fan_speed_layer at more than 1, the first layer has some fan speed.
 . updated fan speeds that cannot be increased even with low layer time (top solid, ironing, support, support interface)
 . updated fan speeds that cannot be reduced by full_fan_speed_layer (bridge, infill bridge, overhangs, support interface)
 . updated extrusion speed that can't be reduced (overhang, bridge)
 . updated extrusion speed that shouldn't be reduced if possible (external, ironing, infill bridge, thin wall, support interface)
 . better name for CoolingBuffer::process_layer variables

#3735
#3608
#3568
#3415
#3408
#3380
#3367
#3361
#3271
#3152
#2872
#2696
#2419
#2326
legend069 pushed a commit to legend069/SuperSlicer that referenced this issue Nov 19, 2023
 . now easier to add new control and do complicate stuff in the cooling buffer.
 . less overall number of lines of code for better results.
 . add 6 new fan control.
 . now if the disable_fan_first_layers is at 0 and full_fan_speed_layer at more than 1, the first layer has some fan speed.
 . updated fan speeds that cannot be increased even with low layer time (top solid, ironing, support, support interface)
 . updated fan speeds that cannot be reduced by full_fan_speed_layer (bridge, infill bridge, overhangs, support interface)
 . updated extrusion speed that can't be reduced (overhang, bridge)
 . updated extrusion speed that shouldn't be reduced if possible (external, ironing, infill bridge, thin wall, support interface)
 . better name for CoolingBuffer::process_layer variables

supermerill#3735
supermerill#3608
supermerill#3568
supermerill#3415
supermerill#3408
supermerill#3380
supermerill#3367
supermerill#3361
supermerill#3271
supermerill#3152
supermerill#2872
supermerill#2696
supermerill#2419
supermerill#2326
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

No branches or pull requests

3 participants