Greifent / afc-gui

GUI for the asus-fan-control project
MIT License
38 stars 5 forks source link

No CPU temp #17

Closed mews-se closed 2 years ago

mews-se commented 2 years ago

Describe the bug afc-gui can't display cpu temp despite having all dependencies. ux430ua

Screenshots Screenshot at 2022-06-25 00-45-51

Greifent commented 2 years ago

Hello, try running the get_current_cpu_temp command and see what it reports

mews-se commented 2 years ago

Hello, try running the get_current_cpu_temp command and see what it reports

Traceback (most recent call last): File "/usr/local/bin/get_current_cpu_temp", line 8, in <module> sys.exit(get_current_cpu_temp()) File "/usr/local/lib/python3.8/dist-packages/AvgCpuTemp/avg_cpu_temp.py", line 25, in get_current_cpu_temp for stamp, value_in_c in itertools.islice(zip(timestamp, sensor), 1): File "/usr/local/lib/python3.8/dist-packages/AvgCpuTemp/avg_cpu_temp.py", line 14, in __next__ return psutil.sensors_temperatures()['k10temp'][0][1] KeyError: 'k10temp'

Greifent commented 2 years ago

That seems to be an issue with the dependency, would suggest trying to reinstall, and if its still present try to open an issue on the CPU-temperature-monitor github here, even if the project seems no longer mantained

notquitethereyet commented 2 years ago

i have the same issue :(

Greifent commented 2 years ago

That is weird, i guess that reinstalling the dependency didnt help, since its the 3rd time issues with this dependency has been reported (probably the fact that the last update was 4 years ago doesnt help), ill look into another way to have the temperature displayed.

Greifent commented 2 years ago

Ok so, I've removed the dependency, unfortunately for now how it works it's the same, I'm still trying to find another solution, it's turning out more difficult than I thought

Greifent commented 2 years ago

Quick update, probably found a way, if you could please update to version 2.0.1 (if you are using gitpack just re-run the install command) and let me know if it's fixed or not, thanks!

notquitethereyet commented 2 years ago

what about the afc-gui-gtk4.cmb file? does it need to moved somewhere? also showingthis error image

Greifent commented 2 years ago

that file is not necessary, and my bad, forgot to change a variable, ill fix it now

Greifent commented 2 years ago

Should be fixed

notquitethereyet commented 2 years ago

seems to work fine. Thanks for the quick update image

Greifent commented 2 years ago

No worries, ill wait for @mews-se to report then ill close the issue

notquitethereyet commented 2 years ago

cd afc-gui/src/ sudo rm /usr/bin/afc-gui sudo rm -r /usr/share/afc-gui/ sudo rm /usr/share/applications/afc-gui.desktop

sudo cp afc-gui /usr/bin/ sudo mkdir /usr/share/afc-gui sudo cp afc-gui-gtk4.ui /usr/share/afc-gui/ sudo cp afc-gui.desktop /usr/share/applications/

also can you add these commands, it took my autistic brain quite some time to understand the instructions lol

Greifent commented 2 years ago

Yea sure, ill add them now

EDIT: I've updated the README.md

mews-se commented 2 years ago

Thanks for a quick fix. Unfortunatly now I'm stuck needing GTK 4 so it doesn't run at all :)

Greifent commented 2 years ago

I didnt thought of that possible issue, i can pass you the gtk3 version, i still have it, just need to update the temperature stuff

Greifent commented 2 years ago

I uploaded the gtk3 version here https://github.com/Greifent/afc-gui-gtk3/

mews-se commented 2 years ago

Yay. Thanks!

Screenshot at 2022-07-09 21-54-53

Greifent commented 2 years ago

No worries, i will close the issue now