Bug 41611 – ksensors - excessive precision for temperature and fan readings in panel

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Paul Taylor 2004-02-14 17:58:13 UTC]

After upgrading to KDE 3.2, Qt 3.3.0 and KSensors 0.72, I've noticed that the 
temperature and fan readings are now shown with six decimal places (all zeroes.)
Previously only the integer portion was displayed; the new format makes the fan 
readings impossible to read.  The two decimal place precision of the voltage 
sensors is displayed correctly.


Reproducible: Always
Steps to Reproduce:
1.  Run KSensors.
2.  Open the panel.
Actual Results:  
Temperature and fan sensors are rendered with six decimal places e.g. "59.000000"


Expected Results:  
Integer values only e.g. "59"

Portage 2.0.50-r1 (default-x86-1.4, gcc-3.3.2, glibc-2.3.3_pre20040207-r0,
2.6.3-rc2-gentoo)
=================================================================
System uname: 2.6.3-rc2-gentoo i686 AMD Athlon(tm) XP 1800+
Gentoo Base System version 1.4.3.13
Autoconf: sys-devel/autoconf-2.59
Automake: sys-devel/automake-1.8.2
ACCEPT_KEYWORDS="x86 ~x86"
AUTOCLEAN="yes"
CFLAGS="-march=athlon-xp -O2 -pipe"
CHOST="i686-pc-linux-gnu"
COMPILER="gcc3"
CONFIG_PROTECT="/etc /usr/X11R6/lib/X11/xkb /usr/kde/2/share/config
/usr/kde/3.1/share/config /usr/kde/3.2/share/config /usr/kde/3/share/config
/usr/share/config /var/qmail/control"
CONFIG_PROTECT_MASK="/etc/gconf /etc/env.d"
CXXFLAGS="-march=athlon-xp -O2 -pipe"
DISTDIR="/usr/portage/distfiles"
FEATURES="autoaddcvs ccache sandbox"
GENTOO_MIRRORS="http://mirror.pacific.net.au/linux/Gentoo"
MAKEOPTS="-j2"
PKGDIR="/usr/portage/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
PORTDIR_OVERLAY=""
SYNC="rsync://rsync.au.gentoo.org/gentoo-portage"
USE="3dnow X acpi alsa apm arts avi berkdb cdr crypt cups dvd encode fbcon
foomaticdb freetype gdbm gif gpm gps gtk gtk2 imlib java jpeg kde libg++ libwww
mad mikmod mmx motif mozilla mpeg ncurses nptl oggvorbis opengl pam pdflib perl
png ppds python quicktime readline sdl slang spell sse ssl svga tcpd truetype
usb x86 xml2 xv zlib"

[Comment 1 Harm Geerts 2004-02-17 15:38:48 UTC]

The cpu speed display is also broken.
It just shows 0

Also with KDE 3.2.0, qt-3.3.0 and ksensors-0.72

[Comment 2 Harm Geerts 2004-03-07 10:45:10 UTC]

Created attachment 26994 [details, diff]
fixes excessive precision and cpu panel

Finally had some time to check this out.
Simple fix but I'm not sure why this never caused any trouble before.

[Comment 3 Dominik Stadler (RETIRED) 2004-03-08 03:40:41 UTC]

Can you send that patch also upstream to http://bugs.kde.org/, please?

[Comment 4 Harm Geerts 2004-03-08 10:56:43 UTC]

Correct me if I'm wrong but I thought ksensors was not maintained by the kde crew
I can submit the patch to ksensors on sourceforge though

[Comment 5 Harm Geerts 2004-03-13 11:18:03 UTC]

Created attachment 27300 [details, diff]
ksensors-proposed-fixes.patch

includes fixes from the patch above and also adds support for lm90 temp sensor.

There might be more sensors that benefit from this update.

[Comment 6 Harm Geerts 2004-03-13 11:22:09 UTC]

Created attachment 27301 [details]
ksensors-0.7.2-r1.ebuild

ebuild to go with the patch

[Comment 7 Paul Taylor 2004-03-13 15:17:11 UTC]

The new patch and ebuild works for me - thanks.

(For the record, I'm now using KDE 3.2.1 and Qt 3.3.1-r1.)

[Comment 8 Martin LORANG 2004-04-10 08:34:06 UTC]

The patch works for me too on amd64.

[Comment 9 Danny van Dyk (RETIRED) 2004-04-11 18:18:58 UTC]

I already keyworded the ebuild ~amd64. KDE Team, please include this patch for
amd64, too.

[Comment 10 Dominik Stadler (RETIRED) 2004-05-01 05:40:08 UTC]

I have now added Version 0.72-r1 to Portage with keywords ~x86 and ~amd64, please test.

[Comment 11 Paul Taylor 2004-05-01 16:59:24 UTC]

Version 0.72-r1 works for me - nice work!  (The earlier patch also worked fine,
but I was lazy and didn't reinstall it after KDE 3.2.1 and 3.2.2 were released.)