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

Create Asus N550JX.xml #574

Merged
merged 2 commits into from
Nov 29, 2019
Merged

Create Asus N550JX.xml #574

merged 2 commits into from
Nov 29, 2019

Conversation

mguludag
Copy link
Contributor

this config based on asus n550jv config but the jx model's cpu is produce more heat so increased fan speed at lower temps

this config based on asus n550jv config but the jx model's cpu is produce more heat so increased fan speed at lower temps
@erkexzcx
Copy link
Contributor

erkexzcx commented Oct 16, 2018

Asus N550JX does not sound like brand new laptop. If it's older than 3-4 years (from manufacturing date) - you must replace thermal compound for both GPU/CPU, clear dust from the motherboard/heatsinks/fans and try again. I just simply cannot trust your "jx model's cpu is produce more heat".

Also understand that NBFC does not monitor GPU temperature. It monitors only CPU temperature, so using NBFC for "better cooling" for graphics intensive tasks, where secondary GPU is used, is not a solution. You should better make profile which fixes the most common issue - fan always on. This is exactly how "Asus N550JV" profile works - I had this laptop year ago and profile worked just great.

EDIT: Hot CPU usually heats up the whole heatspreader, which is shared between fan, CPU and GPU. See this as well for a reference: daringer/asus-fan#47

@hirschmann
Copy link
Owner

@cheytacllc
Thank you very much for your effort.
I'd love to merge your pull request, but currently it doesn't pass all the unit tests:

Asus N550JX config is not valid:

  • All up-threshold values must be lower than the critical temperature
    --> At least one up-threshold is higher than or equal to the critical temperature: 70

Expected: True
Actual: False

see: https://ci.appveyor.com/project/hirschmann/nbfc/builds/19470464/tests

Please fix this small issue and update the pull request, so I can merge it :)

@hirschmann hirschmann merged commit 3ca34d2 into hirschmann:master Nov 29, 2019
vdurante pushed a commit to vdurante/nbfc that referenced this pull request Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants