-
Notifications
You must be signed in to change notification settings - Fork 67
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
2nd Threshold? #44
Comments
Hey, why not but it would need some more code and I don't have much time actually. It will also need a second
So that no 2nd threshold would be needed and it would (should) fit any CPU configuration. |
Yeah, that sounds perfect. |
I would love to see this as well. If possible adding a few thresholds and making this more variable would be great. |
I would love to see this. if I have the time, I will try to put it together and commit for review. |
Having another step would be nice or even a curve that could be calculated based on the FAN_SPEED variable, it would still require a second variable but it could work something like: So it will run at 10% until it hits 45 degrees, then it will increment fan speed to try to bring it down before it hits the threshold of 50 and goes full blast. |
wrote this before seeing issue tigerblue77#44 and tigerblue77's comment on implementation
I had a go at creating a pull request for this. |
Hi, I created pull request for that upgrade. I used line interpolation in my solution. You can check it in #60. Quick example how my solution work:
|
It might be nice if there were an intermediate threshold as well. For example, I have mine set at 25% fan speed, which is fine most of the time. However, occasionally when the load increases, it will jump to the BIOS setting, and be running at like 93% for a bit until it cools down.
I'd prefer to have at least one more threshold, as an option. When it hits that 2nd threshold, perhaps I'd set the fan to go up 45% or 50% instead of the BIOS settings. Then, if it hits my current threshold, it would kick over to the BIOS like normal.
Hopefully that makes sense, and I know that if you could have 2, then people could request 3, 4, or 5. Hopefully people wouldn't ask for that, but I think that 2 would be sufficient for me.
Either way, thanks a ton. This has been the best solution I've found.
The text was updated successfully, but these errors were encountered: