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

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

Open
hardwareadictos opened this issue May 13, 2020 · 28 comments · May be fixed by #69
Open

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

hardwareadictos opened this issue May 13, 2020 · 28 comments · May be fixed by #69

Comments

@hardwareadictos
Copy link

hardwareadictos commented May 13, 2020

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

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

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

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

@hardwareadictos hardwareadictos changed the title Applet crashing rear opening Thermal Monitor Settings [plasma-desktop 5.18.5-2] Applet crashing rear opening Thermal Monitor Settings May 14, 2020
@imatimba
Copy link

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

@hardwareadictos
Copy link
Author

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

imatimba commented May 16, 2020

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

Ashark commented May 18, 2020

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

imatimba commented May 21, 2020

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

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

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

imatimba commented May 21, 2020

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

jnines commented May 21, 2020

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

acutbal commented May 22, 2020

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

@XinqiBao
Copy link

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

@valdotdev
Copy link

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

@mariocarbone
Copy link

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

Do you know how to configure it without menu?

@roachsinai
Copy link

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

mruiz42 commented Jun 1, 2020

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

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

@veganvelociraptor
Copy link

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

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

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

@jnines
Copy link

jnines commented Jun 16, 2020

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

@jnines would you mind sharing the script you used?

@ghost
Copy link

ghost commented Jun 20, 2020

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

jnines commented Jun 20, 2020

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.

@zhimsel zhimsel linked a pull request Jun 29, 2020 that will close this issue
@tkalfaoglu
Copy link

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

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.