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

[4.2.1] Creality stock profile has a button to force fan speed at 100% #6244

Closed
Liger0 opened this issue Aug 23, 2019 · 3 comments
Closed

[4.2.1] Creality stock profile has a button to force fan speed at 100% #6244

Liger0 opened this issue Aug 23, 2019 · 3 comments
Labels
Status: Needs Info Needs more information before action can be taken. Type: New Feature Adding some entirely new functionality.

Comments

@Liger0
Copy link

Liger0 commented Aug 23, 2019

Application version
4.2.1

Platform
Win10 x64

Printer
Cr-10S

Reproduction steps
Just select a stock profile

Actual results
On the fan speed, there is a button to revert to a suggested 100% value instead of the material one

Expected results
There should be no button for it.

Additional information
That button is very dangerous, because it has no sense as you can force the profile if you want to by yourself, but once clicked that button and saved the profile, the material fan speed is not used anymore and you can't change it back to the material one.

@Liger0 Liger0 added the Type: Bug The code does not produce the intended behavior. label Aug 23, 2019
@Ghostkeeper
Copy link
Collaborator

Ghostkeeper commented Aug 23, 2019

Yeah the fx button reverts it to the most recent formula. This formula for the fan speed sets it to 100% if the fan is enabled or 0% if the fan is disabled. A bit of a weird one since the cool_fan_enabled setting is not a parent setting for the fan speed. But right now it's expected behaviour.

We've been considering to remove the button for all settings and maybe replace it with a button to remove it from the custom profile. That's the use case it was originally intended for anyway. That would solve this UX problem as well.

@Ghostkeeper Ghostkeeper added Type: New Feature Adding some entirely new functionality. Category: UX and removed Type: Bug The code does not produce the intended behavior. labels Aug 23, 2019
@GregValiant GregValiant added the Status: Needs Info Needs more information before action can be taken. label Nov 24, 2024
@GregValiant
Copy link
Collaborator

The post-processor Advanced Cooling Fan Control (in Cura since 5.7.1) gives much finer control of the fan speeds.
Can this Feature Request be closed?

Copy link
Contributor

github-actions bot commented Dec 9, 2024

This issue has been automatically closed because there has been no response to our request for more information from the original author. With only the information that is currently in the issue, we don't have enough information to take action. Please reach out if you have or find the answers we need so that we can investigate further.

@github-actions github-actions bot closed this as completed Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Info Needs more information before action can be taken. Type: New Feature Adding some entirely new functionality.
Projects
None yet
Development

No branches or pull requests

4 participants