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

Silent Fan profile added (v3.0) #151

Closed
wants to merge 6 commits into from

Conversation

Adravil
Copy link
Contributor

@Adravil Adravil commented Sep 8, 2023

Option Silent4000RPMThreshold in FanControl.dll.ini now defines at what temperature Silent fan curve should reach 4000 RPM. Minimum -- 80, default -- 85, maximum -- 90.

More tests are welcomed of course, but according to my tests:

  • in Baldur's Gate 3 with any heavy graphical settings
  • with Silent4000RPMThreshold being set to default value (85)
  • with Silent fan profile-curve activated
  • and TDP set to 15W

SoC's temperature stays at ~90°C, what pushes the fan up to 6450 RPM and leaves some headroom for sudden jumps (to ~95°C, never seen 100°C). In this scenarion noise level is high, but that's expected because of the 15W TDP level.

  • If you think that's still too risky, we can drop the default Silent4000RPMThreshold value to 80.
  • With Silent4000RPMThreshold kept at 85, the TDP limitation to 12W lowers temperatures by 5°C (average under stress from ~90°C to ~85°C, sudden jumps from ~95°C to ~90°C). Such change eliminates most of the noise heard in previous scenario (Silent4000RPMThreshold == 85, TDP == 15W).
  • Lowering the TDP level to 10W eliminates the noise almost completely.

ayufan and others added 6 commits September 2, 2023 11:23
…ctions in gamepad mode (ayufan#143)

* Update SteamButtons.cs

removed deadzone and min change

* Update SteamButtons.cs

Adjusted deadzones to minimise snapping to cardinals in xinput mode while attempting to simultaneously avoid drifting of the mouse cursor or scroll in the desktop mode
@Adravil Adravil marked this pull request as ready for review September 8, 2023 10:49
ayufan pushed a commit that referenced this pull request Sep 24, 2023
@ayufan
Copy link
Owner

ayufan commented Sep 24, 2023

@Adravil Thank you. I merged this in here with small amendments: 0518e30

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

Successfully merging this pull request may close these issues.

3 participants