kotelnik / plasma-applet-thermal-monitor

Plasma 5 applet for monitoring CPU, GPU and other available temperature sensors.
GNU General Public License v2.0
111 stars 63 forks source link

[plasma-desktop 5.18.5-2] Applet crashing rear opening Thermal Monitor Settings #67

Open hardwareadictos opened 4 years ago

hardwareadictos commented 4 years ago

Rear updating my arch something broke the Widget. Done it on firstly on my laptop and did it two days later on my Desktop (same Arch installation on both) with the same results.

I'm going to upload a log and

the packets that got updated (because i don't know which one caused the breakage).

thermal_monitor_journalctl.txt

package_update.txt

I already tested other widgets and none of them are having issues.

Plasma version: plasma-desktop 5.18.5-2

Let me know if you need more info :)

TimoWilken commented 4 years ago

I installed plasma5-applets-thermal-monitor from the Arch Linux community repository (and nothing else; I had already installed its dependencies). Adding the applet to the plasma bar works, but trying to configure the applet crashes plasmashell, even after a reboot.

hardwareadictos commented 4 years ago

I installed plasma5-applets-thermal-monitor from the Arch Linux community repository (and nothing else; I had already installed its dependencies). Adding the applet to the plasma bar works, but trying to configure the applet crashes plasmashell, even after a reboot.

That's my exact issue. I even tried to install it via KDE store and via KDE applet application.

rohitghali commented 4 years ago

Even I tried installing it from everywhere. Same issue. Best applet. Please fix it.

imatimba commented 4 years ago

Bug report on KDE's side: https://bugs.kde.org/show_bug.cgi?id=421392

hardwareadictos commented 4 years ago

Bug report on KDE's side: https://bugs.kde.org/show_bug.cgi?id=421392

Nice! But I think that's not caused by Nvidia driver update. I suspect more about plasma lts update imo

imatimba commented 4 years ago

Bug report on KDE's side: https://bugs.kde.org/show_bug.cgi?id=421392

Nice! But I think that's not caused by Nvidia driver update. I suspect more about plasma lts update imo

Yeah, when I saw this bug report just now I did a new comment disregarding the info about the gpu/drivers

Ashark commented 4 years ago

There is also a Fan Speed Monitor applet, which says in Readme that it is based on this widget, and it also crashes when trying to configure widget.

imatimba commented 4 years ago

Well, the kde bug report was marked as duplicate of a generic 4 year old bug report that no one has even look at, it seems. 15 bug reports were marked as duplicates of this old bug report already. Not sure what to think about it. It looks like a WONTFIX to me.

hardwareadictos commented 4 years ago

Well, the kde bug report was marked as duplicate of a generic 4 year old bug report that no one has even look at, it seems. 15 bug reports were marked as duplicates of this old bug report already. Not sure what to think about it. It looks like a WONTFIX to me.

Well, that's not a KDE issue, it's an applet issue. We won't get any support outside here...

akien-mga commented 4 years ago

Well, that's not a KDE issue, it's an applet issue. We won't get any support outside here...

I beg to differ, if a plasmoid can crash plasmashell, that's a bug in plasmashell.

The issue can likely be worked around in the plasmoid to avoid making plasmashell crash, but the latter should not be crashable by user plugins in the first place.

imatimba commented 4 years ago

Well, the kde bug report was marked as duplicate of a generic 4 year old bug report that no one has even look at, it seems. 15 bug reports were marked as duplicates of this old bug report already. Not sure what to think about it. It looks like a WONTFIX to me.

Well, that's not a KDE issue, it's an applet issue. We won't get any support outside here...

Considering the applet was working fine until now it's possible that the bug is on KDE's side.
Unless they've changed some internal behavior on purpose that makes code in this applet "the" bug.

edit: posted new comment by mistake. I was thinking on doing a git bisect, but considering the complexity and size of KDE I wouldn't know where to start. The bug could be on plasma-workspace, plasmashell, plasma-framework, etc.

jnines commented 4 years ago

For the time being I'm just using a command in the Command Output plasmoid to get my CPU/GPU temps. I don't know if kotelnik is still active or not, or if one of the numerous forks has it fixed.

acutbal commented 4 years ago

Hi!! I also have this issue on my computers, every time I try to config the thermal widget crashes plasma. Regards.

XinqiBao commented 4 years ago

same issue here. And I found another widget kargos, can run script and works really well for me. Thanks @roachsinai shared this solution, even it was for another issue.

mariocarbone commented 4 years ago

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 thermal icon and i can't open the config menu :(

valdotdev commented 4 years ago

Same problem on Manjaro 20.0. Crashes when trying to access the settings.

mariocarbone commented 4 years ago

Same problem on Manjaro 20.0. Crashes when trying to access the settings.

Do you know how to configure it without menu?

roachsinai commented 4 years ago

Same problem on Manjaro 20.0. Crashes when trying to access the settings.

Do you know how to configure it without menu?

Should be in ~/.config/plasma-org.kde.plasma.desktop-appletsrc.

mruiz42 commented 4 years ago

I beg to differ, if a plasmoid can crash plasmashell, that's a bug in plasmashell.

The issue can likely be worked around in the plasmoid to avoid making plasmashell crash, but the latter should not be crashable by user plugins in the first place.

Agreed. Here is the output for mine. It seems to crash right after detecting the disks for me. Not sure what the debug should look like when it opens the config window under normal conditions. https://pastebin.com/D0AUaLPT

hardwareadictos commented 4 years ago

Hey! Today i updated my Fedora 32 Workstation and that bug appeared on KDE also.

veganvelociraptor commented 4 years ago

I have the same issue here using Arch and Manjaro. It can't be related to any nvidia driver issue, because i don't have any nvidia card nor drivers installed.

BullShark commented 4 years ago

I installed plasma5-applets-thermal-monitor from the Arch Linux community repository (and nothing else; I had already installed its dependencies). Adding the applet to the plasma bar works, but trying to configure the applet crashes plasmashell, even after a reboot.

This issue also affects me in Manjaro. I tried compiling the source from git. I tried installing the AUR package. I had the same problems with both.

bernharl commented 4 years ago

Same here, on Manjaro. Installed the AUR package that was newly updated. It crashes my desktop when trying to convigure.

jnines commented 4 years ago

The problem is upstream bug report and who knows if it'll be wontfix. I've moved on to kargos, requires a little scripting but you do a lot more with it.

moconnor1234 commented 4 years ago

@jnines would you mind sharing the script you used?

ghost commented 4 years ago

In the meantime, here's mine kargos one (I also use icons-in-terminal for the glyphs). You probably want to adapt yours, since I have my own named stuff in lm_sensors:

cputemp=$(sensors | grep 'CPU' | awk '{total += $3; count++} END {print total/count}')
gputemp=$(nvidia-smi --query-gpu=temperature.gpu --format=csv,noheader)
sdatemp=$(sudo /usr/bin/hddtemp /dev/sda | cut -c 34-35)
sdbtemp=$(sudo /usr/bin/hddtemp /dev/sdb | cut -c 31-32)

# Status
echo "<font size='2'>  ${cputemp}°&nbsp;  ${gputemp}°&nbsp;¹ ${sdatemp}°&nbsp;² ${sdbtemp}°| font= SF Pro Display:style=Medium size=14"
jnines commented 4 years ago

I use a few different scripts separated out for different things:

temp=$(sensors | grep -oP 'Tdie.*?\+\K[0-9.]+') 
echo "${temp%%.*}° | color=red bash='/usr/bin/ksysguard' onclick=bash"

for Intel you'd swap Tdie with Package.

gpu_temp=$(nvidia-smi --query-gpu=temperature.gpu --format=csv,noheader)
echo "${gpu_temp}° | color=green

for Nvidia. I don't have an AMD GPU but I'm assuming it would show up under lm_sensors.

I also use a few others for things like mic, speakers, battery level for headset.

tkalfaoglu commented 4 years ago

For my AMD, I used kargos and this command: sensors|grep edge|cut -d'+' -f2|cut -d' ' -f1