T-Troll / alienfx-tools

Alienware systems lights, fans, and power control tools and apps
MIT License
458 stars 39 forks source link

Fan Curve Displays Only One Curve #303

Closed JazzyJazzjazzy closed 1 year ago

JazzyJazzjazzy commented 1 year ago

This is probably not an issue but a configuration question. I duplicated your curves displayed in your AlienFan instructions. Configured as depicted, I have only one curve displayed and not the second yellow curve. I duplicated the curves in the GPU Thermistor also. Do I need to create curves for all sensors or just the sensor that I am using? Why do I not see the ye curves llow curve? Thank you.

T-Troll commented 1 year ago

Yellow curves are from OTHER sensor - GPU in my case. Green from selected one. So if you don't set GPU sensor, you will only see one.

I can share you .reg file with my full configuration, but keep in mind - i have SSD close to GPU, so i need to cool it down too.

JazzyJazzjazzy commented 1 year ago

I set the GPU curve, you can see this in the photo. I also placed check mark in CPU Thermistor, unchecked GPU on right side in same window, set GPU Thermistor curve and unchecked CPU there. Still only one curve showing.

Yes, please send .reg entry and i will review it. Thank yoou.

T-Troll commented 1 year ago

fans.zip Here it is. But be careful:

Some comments. Fan curves are encoded into Fan-%d-%d variables, there first is Fan number in hardware list (i will change it to fan ID soon), second one is WORD - LOBYTE - index in hardware list, HIBYTE - type: 0 = ESIF, 1 = AWCC, 2 - Disk, 3 - AMD, 4 = OHM

JazzyJazzjazzy commented 1 year ago

Okay, I downloaded your .reg file and changed the NAMES to match what I saw in my registry. I deleted one entry because I did not have that sensor. (Deleted "SensorName-0-7"="CPU Participant (GFX)")

It appears that you only created curves for the two sensors, CPU THERMAL THERMISTOR & GPU INTERNAL THERMISTOR. You checked both CPU &GPU boxes on both.

With the exception of"SensorName-0-2"="SSD Thermistor", all other sensor's check boxes are unchecked. I am guessing that you left this one checked because of your aforementioned SSD.

Question:

  1. If the highest value of each curve is the one that is honored when the program is running, then why do you need the second curve on the above "0-2" sensor?

2.Why can I not check only the CPU check box on CPU THERMAL THERMISTOR and only the GPU check box on the GPU INTERNAL THERMISTOR?

  1. Why do the curves change when I select CPU and then GPU in the right window?

A note, both curves appear on each graph respectively using your .reg file. This was not the case on mine. I include my original .reg, as a text file and your .reg file with the changed names for your reference. Thank you.

My Sensors.txt

Your file with My Changed names.txt

T-Troll commented 1 year ago
  1. Because my SSD near GPU and fried by it. So i need to cool it down, if hot, or it starts throttle.
  2. Because you have ONE, JOINED, cooling system. So spin up GPU fan for CPU-only and CPU fan for GPU-only helps to cool it down.
  3. Because curves in window represent active FAN, selected into right list. For ALL sensors, which control it. Historically, it was opposite - curves represent SENSOR for all FANS. But later i reverse the logic as less precise.

Your control have 1 sensor for CPU fan, and 2 for GPU. So it's OK, one curve if CPU fan selected, 2 if GPU fan.

JazzyJazzjazzy commented 1 year ago

Understood. Kindly CLOSE this topic if you wish, thank you.

T-Troll commented 1 year ago

You give me an idea how to modify UI to make this clear for user. So i keep it open until i do it.

JazzyJazzjazzy commented 1 year ago

Sounds good sir. May I offer that the curves that are yellow have the corresponding sensor label displayed as a possible solution?

Alternatively, you could consider making the curves and/or balls different colors that correspond to each sensor. The Sensor entry/name could be made to match the color.

I am sure that you have a simply Grand idea in store for us. Thank you

T-Troll commented 1 year ago

Labels is not so easy to implement good way. I plan to change UI the other way - checkboxes will be not on fan, but on sensors. So you will see selected sensors after select fan.

T-Troll commented 1 year ago

Check latest release.

JazzyJazzjazzy commented 1 year ago

Hello sir, I have the latest release. It is a much clearer interface and easier to work with it. It is easy to see all the individual sensors that are in effect for our curves. Thank you!

T-Troll commented 1 year ago

Thanks. As i said, initially this was fan-driven, later i made it sensor-driven but don't change UI. Anyway, thanks for your questions, this help to clarify.

JazzyJazzjazzy commented 1 year ago

Sir, YOU Are The Man! Peace.