search

crystalidea / macs-fan-control

Control fans on Apple computers
https://crystalidea.com/macs-fan-control
840 stars 97 forks source link

Safer fan speed control based on sensor with highest temperature value #572

Open tbenko73 opened 2 years ago

tbenko73 commented 2 years ago

I suggest adding a new computed property to the sensor values named: Highest temperature This value should be te highest value from temperatures detected on all sensors in the computer. The user should be allowed, to select Highest temperature when choosing to control the fan with Sensor-based value. The user should be allowed to choose in preferences to display Highest temperature in the menubar.

Reason: The current custom setting possibilities, despite the multitude of sensor values available do not seem to offer a really safe choice. Picking the wrong sensor in the wrong situation can easily lead to an overheated computer... It even seems impossible to pick a right sensor in my case where i have a MacBook Pro. If control is made based on CPU sensors, leaves me with the possibility of GPU overheat... If control is made based on GPU sensors, leaves me with the possibility of CPU overheat...

zecanard commented 2 years ago

This and #565 are the most important features right now. Recently I started doing work that uses GPU compute, and I couldn’t understand why my computer was stuttering so badly, eventually leading to a complete lockup and spontaneous reboot. Macs Fan Control only shows one temperature in the menu bar, so at first glance, overheating was not the issue. Turns out I’d been running with the same Macs Fan Control profile for so long, I didn’t think to switch to my “Gaming” profile for GPU work.

But since I was alternating work that used either the CPU or the GPU, my best option was to return to Auto control, despite the slower fan ramp-up (and the resulting throttling). This in turn made my Pro license (and indeed, the entire app) rather useless.

But even when monitoring “the CPU,” modern computers all have multiple cores. The Average sensor isn’t very useful it when 1 core could be significantly hotter than the other 7 (or more).

Please consider bumping up the priority on these features.

hastryn commented 1 year ago

Bump!

As a hardware engineer who knows how heat is the #1 enemy of computers I feel your product is great! And I see adding this "Max" feature would be a huge benefit - I would personally have all the family members that I provide IT support for buy your software if you add this feature!

And once the Max feature is implemented taking the code to the next level would would be to allow us to write our own equations to create "custom" profiles (which I admit requires a much more detailed discussion on the feature definition and practical implementation to make sure people don't break their systems when using it.)