No GPU temps (Nvidia) after updating to 0.10

[t-kowalewski]

Checklist

• I've looked through the troubleshooting docs, the known problems list, and existing open issues for similar issues.

What operating system and version are you using?

EndeavourOS - 6.6.44-3 LTS

What architecture are you using?

x86_64/AMD64

What terminal(s) are you running bottom on that are experiencing the problem?

Konsole 24.05.2

What filesystem(s) are you using?

ext4

What version of bottom are you running?

0.10.2

How did you install bottom?

official Arch repo

Describe the issue

bottom stopped displaying GPU temperature after updating it from version 0.9.6
Used GPU - mobile Nvidia GTX 660m

What is the expected behaviour?

bottom showing GPU temperature in according section

What is the actual behaviour?

bottom doesn't display GPU temps

How can we reproduce this?

Run bottom on an Arch-based system with discrete mobile GPU from Nvidia

Additional information

No response

[ClementTsang]

Hmm, I can't easily troubleshoot this (no Nvidia GPU) but I'll take a look. My AMD card shows temperatures fine.

[ClementTsang]

One thing that would be great if you check for me first - if you expand the temperature table (e) does it show up on the list? Also:

mobile

Hm, if you make the dGPU do some work, does it show up? Just curious as there is some code to purposely avoid checking temperatures of asleep devices to avoid constantly waking them up - in that case, it will just be given a temperature of "N/A", though it should still be on the list.

[t-kowalewski]

Thanks for the reply. No, unfortunately extended view doesn't show it either.

Granted, it's a pretty old GPU running on legacy drivers, but the sensor still outputs the information as expected. Just not in bottom :/

[ClementTsang]

Hm, weird. I did bump nvml-wrapper in between updates so it's possible something changed between versions, I'll see if I can figure it out. Thanks for the extra info.

[ClementTsang]

Also, was it properly showing a temperature reading in 0.9.6 (e.g. not just "N/A" or something empty)? And if it's not too much trouble, could you show it? Thanks.

[t-kowalewski]

Downgraded to 0.9.7 and GPU temp is back. Thanks again for looking into this.

[shurizzle]

The "issue" (apparently, it’s intended behavior) was introduced by commit e4a6e75 (issue #1276). By default, all GPU information is disabled, so you need to enable it using the enable_gpu flag. Perhaps it would be better to enable it by default?

[ClementTsang]

🤦 yeah that would do it, I forgot we had a change that bundled that all together. I had thought we had changed it to enable GPU info by default, but I guess that slipped through the cracks and I misremembered. I'll look into changing that.

Thanks for figuring out the problem!