plasma-applet-thermal-monitor icon indicating copy to clipboard operation
plasma-applet-thermal-monitor copied to clipboard

CPU temperature goes to off state

Open tairosonloa opened this issue 6 years ago • 56 comments

I use this applet to show my CPU, GPU, SSD and HDD temperature. However, I have a problem with the CPU temperature.

When I start the computer, all the temperatures show its values except CPU, which shows an "off" message. Also, when I'm using my computer, sometimes I realize CPU temperature is "off" again. This can be manually fixed by right clicking on the applet and left click on "Reload temperature sources".

My CPU is a AMD Ryzen 5 1500x and I'm using the lmsensors/nct6795-isa-0a20/CPUTIN

I think this can be "patched" if a new feature to auto reload temperature sources on these situations is added.

tairosonloa avatar Dec 16 '18 11:12 tairosonloa

same problem with intel

alium avatar Dec 30 '18 15:12 alium

Also, I saw a day ago an user (Nano-Ocelot) wrote a comment on the kde store. I copy and paste:

I have a weird bug where I have to 'refresh sources' after logon to display cpu core group temperatures (highest temp core shown). Otherwise it remains blank. However gpu temp (single temp source) works correctly on initial widget load.

To see the original comment: https://store.kde.org/p/998915/#ratings-panel

tairosonloa avatar Jan 01 '19 18:01 tairosonloa

Same problem here with Manjaro. Seems something updated and broke the initial temp readings of the widget with CPU. I am on Intel btw. EDIT: Seems this is the same issue as #16

GuerreroNinja avatar Jan 05 '19 20:01 GuerreroNinja

There is same problem I am using Manjaro on Dell Vostro 5568.

ademcal avatar Jan 06 '19 07:01 ademcal

I'm also seeing this issue, Intel CPU and AntergOS. Seems like the problem was caused by some update in archlinux, but really don't know.

emilio-aburto avatar Jan 07 '19 00:01 emilio-aburto

Issue still persists on Arch Linux-LTS Kernel v4.19.14-1, Intel i7 5820k CPU

lvxejay avatar Jan 13 '19 01:01 lvxejay

Same issue here, same kernel and OS as @lvxejay, CPU is Intel i7-7770K.

cryzed avatar Jan 15 '19 11:01 cryzed

Same issue, Manjaro KDE, cup fan shows OFF also.

roachsinai avatar Jan 15 '19 15:01 roachsinai

Can confirm this on Arch Linux as well since updating to the latest major KDE release 5.14. Using the latest git of plasma-applet-thermal-monitor btw.

afettouhi avatar Jan 17 '19 06:01 afettouhi

Same problem, but I found a solution (of sorts). I installed Plasma 5 'Simple System Monitor' which sits on the desktop and I have Thermal Monitor in the panel. Reboot - now Thermal Monitor has the CPU temperature loaded! How this works I do not know, but maybe there's a clue for someone smarter than me to get Thermal Monitor showing CPU temperature at startup, by itself. Arch Linux, LTS kernel 4.19.15-1, Intel i7-7500U

severnaside avatar Jan 19 '19 12:01 severnaside

@severnaside you mean this plasmoid?

roachsinai avatar Jan 19 '19 13:01 roachsinai

@roachsinai Yes

severnaside avatar Jan 19 '19 16:01 severnaside

I think this issue #52 is useful, cause I comment https://github.com/kotelnik/plasma-applet-thermal-monitor/blob/783536952c38bf26aefadb36d2dadd50c8ddc2c0/package/contents/ui/main.qml#L312 thisl line. No need refresh by yourself. But it's not a so good method.

Cause I don't konw how to debug a plasmoid and have not learn javascript(just know a very little), someone maybe could debug the QML type error.

roachsinai avatar Jan 21 '19 03:01 roachsinai

Maybe solve the problem as I found var systemmonitorAvailableSources should be an array, change the code according to var QML Basic Type... Made a pull request #54 , check it and update if you have a better solution.

roachsinai avatar Jan 21 '19 03:01 roachsinai

pull #54 fixed problem with manual reload not, i tested the patch today. After restart the computer came again OFF

alium avatar Jan 21 '19 23:01 alium

@alium What's your Linux Distro? I only test it on Manjaro. For me there are 3 way to let cpu OFF disappear.

  1. comment systemmonitorAvailableSources.push(source) at plasma-applet-thermal-monitor/package/contents/ui/main.qml line 312
  2. change property var systemmonitorAvailableSources to property var systemmonitorAvailableSources: [] at plasma-applet-thermal-monitor/package/contents/ui/main.qml line 58
  3. pull #54

So, I think maybe you could try others.

And make sure after you pull #54 , you also update code at /usr/share/plasma/plasmoids/org.kde.thermalMonitor (maybe some difference according to Distro). Or just update manually.

So sorry that only you reply Not Work, haven't get a Work reply(but 2 thumb up at my previous reply).

roachsinai avatar Jan 21 '19 23:01 roachsinai

@roachsinai i try your instructions, but ended with error: file:///usr/share/plasma/plasmoids/org.kde.thermalMonitor/contents/ui/main.qml:312:1: Expected token `}'

seems as problem with comment "systemmonitorAvailableSources.push(source)"

use Archlinux with OpenRC (also Artix Linux)

alium avatar Jan 22 '19 10:01 alium

First,those 3 method is mutually exclusive.

And for now I think the different between your sittuation an my is I installed systemed version Manjaro. So, I can't reproduce it. And I think the error is wierd.

Do you get the error message on booting?

roachsinai avatar Jan 22 '19 12:01 roachsinai

What I did is try to solve #52 which let got the idea that systemmonitorAvailableSources maybe not support push method. But maybe this is something different in problem, though change systemmonitorAvailableSources to an array is work for me.

roachsinai avatar Jan 22 '19 12:01 roachsinai

@roachsinai oh, ok! now i understand :-) i try all 3 methods

alium avatar Jan 22 '19 12:01 alium

  1. comment systemmonitorAvailableSources.push(source) at plasma-applet-thermal-monitor/package/contents/ui/main.qml line 312
  2. change property var systemmonitorAvailableSources to property var systemmonitorAvailableSources: [] at plasma-applet-thermal-monitor/package/contents/ui/main.qml line 58
  3. pull #54
  1. works not for me, file:///usr/share/plasma/plasmoids/org.kde.thermalMonitor/contents/ui/main.qml:312:1: Expected token `}'
  2. works and fix this bug #53 :-D
  3. fix this bug not

THANKS!!!

alium avatar Jan 22 '19 13:01 alium

@roachsinai

So sorry that only you reply Not Work, haven't get a Work reply(but 2 thumb up at my previous reply).

Yes, pull #54 worked for me. Arch Linux, LTS kernel 4.19.15-1, Intel i7-7500U

severnaside avatar Jan 22 '19 13:01 severnaside

@alium @severnaside glad to hear that.

roachsinai avatar Jan 22 '19 13:01 roachsinai

Applying pull https://github.com/kotelnik/plasma-applet-thermal-monitor/pull/54 to ~/.local/share/plasma/plasmoids/org.kde.thermalMonitor/contents/ui/main.qml (no thermalMonitor in /usr/share/ ) worked for me, too. (using manjaro)

Termuellinator avatar Jan 23 '19 17:01 Termuellinator

Hi everyone, I got same effect with another plasmoid named kargos. And the effect is more better for me that as I use Bumblebee. Will show GPU temperature if I use nvidia and no GPU info completely if I'm not using GPU.

Not using Nvidia: sys_info

Using Nvidia: sys_info_with_gpu

And kargos makes your simple script works like a plasmoid, so I write a bash script.

What's more, kargos is compatible with Bitbar which provide lots of useful scripts. Check it at k-argos-plugins and another two k-argos plugin.

roachsinai avatar Jan 25 '19 17:01 roachsinai

#54 Works for me

sgparry avatar Feb 26 '19 20:02 sgparry

I use kargos with hdsentinel to monitor my HDs health and temperature. Works like charm.

Pull #54 works also for me btw.

GuerreroNinja avatar Feb 26 '19 21:02 GuerreroNinja

DEB-package for bionic (with #54 PR): https://launchpad.net/~varlesh-l/+archive/ubuntu/bionic/+files/plasma-applet-thermal-monitor_1.2.9git20190121_amd64.deb

varlesh avatar Feb 28 '19 03:02 varlesh

@varlesh That package still displays OFF for CPU temps for me in Kubuntu 19.04. I suppose, it means https://github.com/kotelnik/plasma-applet-thermal-monitor/pull/54 doesn't work.

Lastique avatar May 12 '19 01:05 Lastique

@Lastique Worked, but after reboot plasma. Added CPU temp sensor > killall plasmashell > ALT+F2 > plasmashell image

varlesh avatar May 14 '19 19:05 varlesh