zocker-160 / thinkfan-ui

A small gui app for Linux to control the fan speed and monitor temps on a ThinkPad
GNU General Public License v3.0
88 stars 8 forks source link

temp not displayed #5

Closed gnhartch closed 1 year ago

gnhartch commented 1 year ago

just upgraded to latest version and temperature data is missing.

sensors in terminal pulls up data as expected but not displayed in thinkfan-ui application window :/

zocker-160 commented 1 year ago

can you please post the output of the sensors command here?

gnhartch commented 1 year ago
[buttx@tartarus ~]$ sensors
coretemp-isa-0000
Adapter: ISA adapter
Package id 0:  +53.0°C  (high = +86.0°C, crit = +100.0°C)
Core 0:        +53.0°C  (high = +86.0°C, crit = +100.0°C)
Core 1:        +48.0°C  (high = +86.0°C, crit = +100.0°C)

BAT0-acpi-0
Adapter: ACPI interface
in0:          12.19 V  

thinkpad-isa-0000
Adapter: ISA adapter
fan1:        2880 RPM

acpitz-acpi-0
Adapter: ACPI interface
temp1:        +52.0°C  (crit = +99.0°C)
zocker-160 commented 1 year ago

thank you I will look into it

blipk commented 1 year ago

I'm having a similar problem: image

I'm not sure if this is related to @gnhartch issue as they didn't provide a screenshot, I'm also not sure if it's specific to thinkfan or a broken config on my system (conflicting themes and font configuration?) because I can fix it running with this QT env var: QT_QPA_PLATFORMTHEME=qt5ct thinkfan-ui image

@zocker-160

gnhartch commented 1 year ago

all the other information was displayed just not the CPU temp.

I tried solution from @blipk and it didn't change anything

blipk commented 1 year ago

It was an issue with the regex from the last PR, I've opened a fix or discussion here: https://github.com/zocker-160/thinkfan-ui/pull/6

zocker-160 commented 1 year ago

@blipk I cannot reproduce this issue and I do not think I can do anything about this, since I am not enforcing any QT theme nor are any colors hardcoded (except the version number, I will fix that), it should always use the system theme.

From my testing it does look correctly when using dark mode.

zocker-160 commented 1 year ago

thank you for reporting the issue @gnhartch, it should be fixed now, I will push new version in a few minutes.

blipk commented 1 year ago

No worries @zocker-160 I figured it's probably my system, so all good.