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

CPU temperature goes to off state #53

Open
tairosonloa opened this issue Dec 16, 2018 · 56 comments
Open

CPU temperature goes to off state #53

tairosonloa opened this issue Dec 16, 2018 · 56 comments

Comments

@tairosonloa
Copy link

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.

@alium
Copy link

alium commented Dec 30, 2018

same problem with intel

@tairosonloa
Copy link
Author

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

@GuerreroNinja
Copy link

GuerreroNinja commented Jan 5, 2019

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

@ademcal
Copy link

ademcal commented Jan 6, 2019

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

@emilio-aburto
Copy link

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.

@lvxejay
Copy link

lvxejay commented Jan 13, 2019

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

@cryzed
Copy link

cryzed commented Jan 15, 2019

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

@roachsinai
Copy link

roachsinai commented Jan 15, 2019

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

@afettouhi
Copy link

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.

@severnaside
Copy link

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

@roachsinai
Copy link

@severnaside you mean this plasmoid?

@severnaside
Copy link

@roachsinai Yes

@roachsinai
Copy link

I think this issue #52 is useful, cause I comment

systemmonitorAvailableSources.push(source)
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
Copy link

roachsinai commented Jan 21, 2019

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.

@alium
Copy link

alium commented Jan 21, 2019

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

@roachsinai
Copy link

@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 Solution #53: CPU temperature goes to off state #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).

@alium
Copy link

alium commented Jan 22, 2019

@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)

@roachsinai
Copy link

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
Copy link

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.

@alium
Copy link

alium commented Jan 22, 2019

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

@alium
Copy link

alium commented Jan 22, 2019

  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 Solution #53: CPU temperature goes to off state #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 CPU temperature goes to off state #53 :-D
  3. fix this bug not

THANKS!!!

@severnaside
Copy link

@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

@roachsinai
Copy link

@alium @severnaside glad to hear that.

@Termuellinator
Copy link

Applying 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)

@roachsinai
Copy link

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.

@sgparry
Copy link

sgparry commented Feb 26, 2019

#54 Works for me

@GuerreroNinja
Copy link

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

Pull #54 works also for me btw.

@varlesh
Copy link
Contributor

varlesh commented Feb 28, 2019

@Lastique
Copy link

@varlesh Indeed, after a system reboot CPU temps tend to show up, although after a delay. I wasn't sure this behavior was reliable, I've only rebooted a couple of times since I installed the package. Thanks!

@fleamour
Copy link

fleamour commented Jun 10, 2019

sudo nano ~/.local/share/plasma/plasmoids/org.kde.thermalMonitor/contents/ui/main.qml

change:
property var systemmonitorAvailableSources
to:
property var systemmonitorAvailableSources: []

Works under openSUSE Tumbleweed.

@hellbenthorse
Copy link

sudo nano ~/.local/share/plasma/plasmoids/org.kde.thermalMonitor/contents/ui/main.qml

change:
property var systemmonitorAvailableSources
to:
property var systemmonitorAvailableSources: []

Works under openSUSE Tumbleweed.

Can confirm this works under Gentoo also, and I dare say any other Distro for that matter.

@lr222gw
Copy link

lr222gw commented Jul 6, 2019

sudo nano ~/.local/share/plasma/plasmoids/org.kde.thermalMonitor/contents/ui/main.qml

change:
property var systemmonitorAvailableSources
to:
property var systemmonitorAvailableSources: []

Works under openSUSE Tumbleweed.

I just did this on Manjaro, now the widget works without having to reload it! 👍

@lvxejay
Copy link

lvxejay commented Jul 6, 2019

Is this plasma applet unmaintained? Could submit a PR if this fixed it

@msdobrescu
Copy link

Under Sabayon Linux works for kernel version 4 but not for 5, where needs manual reload.

@Rush
Copy link

Rush commented Sep 6, 2019

I am glad I found this thread. It's a very annoying issue. Thanks for the solution.

@Robiocop
Copy link

It works for me under KDE Neon Bionic. Thanks for this tip.

@msdobrescu
Copy link

Hi, do you refer to this poor workaround as tip? It is not intended as tip, it just reveals some kernel version 4 vs version 5 difference as reason of the problem.
I'd rather drop the applet than stick with an old kernel.
Probably, the simplest workaround would be to do some refresh at the applet starting, as it manually does and works.

@fleamour
Copy link

Can anyone submit a pull request?

@roachsinai
Copy link

roachsinai commented Sep 12, 2019

Hum...

In fact, I've already made that pr 7 months ago.

There is an alternative choice: k-argos-plugins.

And maybe the pr title is no good, I will rename it.


Update:

Renamed it as Solution #53: CPU temperature goes to off state #54 .

@Rush
Copy link

Rush commented Sep 12, 2019

@roachsinai k-argos is not an alternative. It seems it's doing a lot more than just temperature. I don't need a Chinese calendar.

@roachsinai
Copy link

@Rush what you need is just sysinfo.3s.sh, each script for a different task(like a plasmoid) with the help of kargos in that repo. Maybe you should first take a peek at kargos. If it is not what you want , just forget it.

@msdobrescu
Copy link

Thermal monitor works for various sensors I have, I use it for CPU and video temperature and for CPU fan speed. It simply needs some update to be able to start the CPU temperature and CPU fan automatically, and possibly some more sensors, but the video temperature works from the start...
I don't see the point of discussing alternatives here.

@roachsinai
Copy link

@msdobrescu Yeah you're right man, so I made #54 when I found solution to solve this problem 7 months ago. And people always first go to issues to find whether there is a solution but not check at pull requests right? What's more, maybe another grammer chage, etc, of next kde version will break it again, and it's hard for a newbie of qml like me to find the error and solve it.

So I just share another way can do same thing or more, sysinfo.3s.sh now support Cpu temp, Gpu temp, Cpu speed, Cpu usage, memory usage. As the maintainer big @kotelnik not in active, for me more choice will be better.

@msdobrescu
Copy link

Indeed, somebody should look at the pull requests when they come, assess and accept it or reject it.
Maybe the maintainer does not work on it anymore? Or has no time? In this case, a fork might be the solution.

@roachsinai
Copy link

Don't know, @kotelnik only made an issue last year. Waiting for the return of @kotelnik and keep going.

@afettouhi
Copy link

@roachsinai I tried out the kargos widget. I can't get it to work. I downloaded the kargos widget and downloaded the sysinfo.3s.sh from the plugin github page and set it up. But it only the widget only says starting... and nothing more

@roachsinai
Copy link

@afettouhi maybe you forgot to add run permission, if so use chmod +x sysinfo.3s.sh.

@afettouhi
Copy link

@afettouhi maybe you forgot to add run permission, if so use chmod +x sysinfo.3s.sh.

Just tried that and still just starting... as output.

@roachsinai
Copy link

@afettouhi try to run sysinfo.3s.sh directly in a terminal, my output is:

hmank ~/./argos » ./sysinfo.3s.sh
50°2400|font=iosevka size=14 color=white
---
CPU: 5.0% |font=iosevka size=14 color=white
MEM: 67.2% |font=iosevka size=14 color=white

Maybe check whether you install font iosevka or change it to font you installed.


What's more, if still not working please make an issue in my repo to keep this thread clean.

@afettouhi
Copy link

afettouhi commented Sep 13, 2019

@roachsinai this the output I get running the sh file.

[af@andre Hentninger]$ ./sysinfo.3s.sh
<u>34</u>°<u></u>&nbsp;<u>29</u>°|font=iosevka size=14 color=white
---
CPU: .1% |font=iosevka size=14 color=white
MEM: 12.7% |font=iosevka size=14 color=white

@roachsinai
Copy link

You said set it up meaning have set the executable path at kargos setting? If not make it at the kargos plasmoid settings.

If still not work please create issue at kargos repo or my plugin repo.
Not suitable for discussion here.

Zardoz89 added a commit to Zardoz89/plasma-applet-thermal-monitor that referenced this issue Mar 4, 2020
@Zardoz89
Copy link

Zardoz89 commented Mar 4, 2020

I make a fork over @agurenko fork with a fix to this issue.

@mariocarbone
Copy link

mariocarbone commented May 26, 2020

Is there any way to try to config it via a file?
I removed it and reinstalled apllied the fix and don't work anyway

This is my latest config
image
Now i have the shitty icon :(

Zardoz89 added a commit to Zardoz89/plasma-applet-thermal-monitor that referenced this issue Nov 29, 2020
@gchamon
Copy link

gchamon commented May 7, 2022

I had ksysguard reinstalled after installing the applet and it seems to have solved the problem for me:

https://bbs.archlinux.org/viewtopic.php?pid=1977673#p1977673

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.