Description of problem: After upgrading from F15 to F16, the applet shows that my CPU temperature is at the freezing point of water. Version-Release number of selected component (if applicable): gnome-shell-extension-cpu-temperature-0-0.3.git1f471c7.fc16.src.rpm How reproducible: Always The applet is likely picking a false reading from sensors: [root@monster ~]# sensors nouveau-pci-0600 Adapter: PCI adapter temp1: +0.0°C (high = +100.0°C, crit = +110.0°C) coretemp-isa-0000 Adapter: ISA adapter Core 0: +34.0°C (high = +87.0°C, crit = +97.0°C) Core 1: +32.0°C (high = +87.0°C, crit = +97.0°C) Core 2: +35.0°C (high = +87.0°C, crit = +97.0°C) Core 3: +29.0°C (high = +87.0°C, crit = +97.0°C) coretemp-isa-0001 Adapter: ISA adapter Core 0: +31.0°C (high = +87.0°C, crit = +97.0°C) Core 1: +30.0°C (high = +87.0°C, crit = +97.0°C) Core 2: +32.0°C (high = +87.0°C, crit = +97.0°C) Core 3: +26.0°C (high = +87.0°C, crit = +97.0°C) The applet appears to be showing that false reading of 0C, and ignores all the eight real temperature sensors from my actual CPU cores. Even if I suppress manually suppress that false sensors, in sensors.conf, so that "sensors" no longer shows it, the applet still picks it up.
Can you please report that bug upstream? https://github.com/RaphaelKimmig/Gnome-Presentation-Mode
The upstream URL is bot correct. The extension has been updated at https://github.com/xtranophilist/gnome-shell-extension-cpu-temperature Please verify if the latest version works and report back.
Pulled the latest version. Works better, but it's showing only four out of the eight available temperature sensors. Peeking at the code, its using the sensor label as a key for an associative array. This version is getting confused because sensors is reporting "Core 0" through "Core 3" temperature, for the first CPU, and "Core 0" through "Core 3" temperature for the second CPU. It's getting confused by multiple sensors with the same name. See above for an example.
Sorry, the URL was wrong https://github.com/xtranophilist/gnome-shell-extension-cpu-temperature/issues If you provide a patch I can add that patch to the package as long that upstream didn't fix it.
There are a number of other issues with the current version. It expects that all temperature sensor labels are five characters in length, etc... The names of temperature sensors are exclusively set in /etc/sensors3.conf, and can be anything, and not always five characters in length; and may not necessarily be named "Core X". Generally, Fedora packages for minor software of this kind tends to avoid extensive patching, and stay closer to upstream. It's better to wait until it's fixed upstream.
Updated the code again. The extension should show all different sensors now. Yes, this extension supposes the strings on sensors3.conf aren't modified. It would be a better idea to read the sensors.conf file first and then look for the pattern accordingly in the sensors output. This is very much in the priority list for future along with user preferences and other things. Thank you.
Created attachment 544827 [details] Typo fix This version doesn't work at all, because of this typo. After fixing this typo, it works. There are two lines in the applet's dropdown for each temperature reading. First one reads "ISA Adapter #:", and the next line has the temperature reading formatted as "CoreN: xxC". It's showing all eight core temperature readings, but I think that combining both lines for each temperature sensor, into a single "ISA Adapter #: CoreN xxC" line would've looked less confusing.
Fixed the typo and the temperature is shown how the sensors gives. Core Temperatures from the same ISA adapter are grouped together like: ISA Adapter N: Core0 : xxC Core1 : yyC Core2 : zzC ... Thanks for reporting.
Created attachment 544904 [details] Full output from sensors, and "sensors -u" Yes, that's much clearer. This fix is probably the best one, at this time, but over the long term, it will be better to work with lm_sensors owners to add an option to sensors that would
Created attachment 544907 [details] Full output from sensors, and "sensors -u" ehh.. accidentally submitted the comment too soon. Continuing with the last comment, anyway, have you considered running "sensors -u" instead of "sensors". The output seems to be much easier to parse. Here's what the full output from "sensors" and "sensors -u" shows for me. In addition to the 8 CPU core temperatures, looks like my hardware has at least eight additional temperature sensors on the motherboard, of which three are disables, and of the remaining five enabled sensors, two have garbage, but I still have three other miscellaneous temperature sensors that appear to work. But this version looks reasonable to me, anyway.
This bug should be marked as resolved/fixed.
gnome-shell-extension-cpu-temperature-0-0.4.gite1777e6.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/gnome-shell-extension-cpu-temperature-0-0.4.gite1777e6.fc16
gnome-shell-extension-cpu-temperature-0-0.4.gite1777e6.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/gnome-shell-extension-cpu-temperature-0-0.4.gite1777e6.fc17
Package gnome-shell-extension-cpu-temperature-0-0.4.gite1777e6.fc16: * should fix your issue, * was pushed to the Fedora 16 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing gnome-shell-extension-cpu-temperature-0-0.4.gite1777e6.fc16' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2012-2451/gnome-shell-extension-cpu-temperature-0-0.4.gite1777e6.fc16 then log in and leave karma (feedback).
gnome-shell-extension-cpu-temperature-0-0.4.gite1777e6.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report.
gnome-shell-extension-cpu-temperature-0-0.4.gite1777e6.fc17 has been pushed to the Fedora 17 stable repository. If problems still persist, please make note of it in this bug report.