How to: Get CPU temp

[ariyako]

@jimp:

Even better:
http://files.pfsense.org/jimp/tempmon/

WIN  ;D  :D

[stephenw10]

@seh2000:

I am not a programmer, but if I replace the "C" with another symbol or take the "C" away then at the next update it is replaced with a "C" I believe this is from the command sysctl dev.cpu.0.temperature as the return is nn.nC so the degree symbol and the C added to the system_information.widget.php is overwritten

It exhibits the same behaviour for me and I'm using mbmon to get the temperature which returns only the temperature, nn.n.

Steve

[Metu69salemi]

Waiting to someone package this or make a widget.

[seh2000]

It exhibits the same behaviour for me and I'm using mbmon to get the temperature which returns only the temperature, nn.n.

Steve

Hi Steve,
Tried to get mbmon to work on my box, but cannot as I would have liked to have the fan RPMs, but core temp will do for now and at least now we know why the ' ° ' goes away (overwritten).
Cheers Steen

[stephenw10]

The coretemp sensor is far more accurate. I would use that if my hardware supported it.
I was pointing out that the ° symbol disappears even though mbmon does not return a 'C' character.
I would guess it's being snipped by some other part of the script.

Steve

[seh2000]

@stephenw10:

The coretemp sensor is far more accurate. I would use that if my hardware supported it.
I was pointing out that the ° symbol disappears even though mbmon does not return a 'C' character.
I would guess it's being snipped by some other part of the script.

Steve

Tx for the feedback I will keep using coretemp.
When I replaced the 'C' with Q, 6, z then each time at the first updated they were replaced with the 'C', but why under mbmon in mbmon does not return a 'C' character - no idea.
Maybe one could strip the 'C' by only allow NN.n in the field and have ° C hardcoded somehow. Well at least I can see the temp right now from one care this been a wish for a long time! Thanks to all here for making that happen!
Cheers Steen

[stephenw10]

Speculating….
There are two functions. One that draws the table and all the static data and fills in the dynamic data, system_information.widget.php. Another function updates the dynamic data and it's that function that needs updating to include a °.
...End speculation.

Steve

Edit: Yes. In /usr/local/www/javascript/index/ajax.js
Add a ° to the updateTemp function like this:

function updateTemp(x) {
if($("tempmeter")) {
$("tempmeter").value = x + '°C';
$("tempwidtha").style.width = x + 'px';
$("tempwidthb").style.width = (100 - x) + 'px';
}

[seh2000]

@stephenw10:

Speculating….
There are two functions. One that draws the table and all the static data and fills in the dynamic data, system_information.widget.php. Another function updates the dynamic data and it's that function that needs updating to include a °.
...End speculation.

Steve

Edit: Yes. In /usr/local/www/javascript/index/ajax.js
Add a ° to the updateTemp function like this:

function updateTemp(x) {
if($("tempmeter")) {
$("tempmeter").value = x + '°C';
$("tempwidtha").style.width = x + 'px';
$("tempwidthb").style.width = (100 - x) + 'px';
}

YIHUUUUUU!  ;D  :D  :)
Steve you are the man!
Works like a dream now.
Now I will try to work on getting all four cores to be displayed.
Cheers Steen

[CNLiberal]

@stephenw10:

Speculating….
There are two functions. One that draws the table and all the static data and fills in the dynamic data, system_information.widget.php. Another function updates the dynamic data and it's that function that needs updating to include a °.
...End speculation.

Steve

Edit: Yes. In /usr/local/www/javascript/index/ajax.js
Add a ° to the updateTemp function like this:

function updateTemp(x) {
if($("tempmeter")) {
$("tempmeter").value = x + '°C';
$("tempwidtha").style.width = x + 'px';
$("tempwidthb").style.width = (100 - x) + 'px';
}

When I make the above change, I get this:

°C

That's after the page refreshes.  This is the code I'm putting in:

function updateTemp(x) {
	if($("tempmeter")) {
		$("tempmeter").value = x + '°C';
		$("tempwidtha").style.width = x + 'px';
		$("tempwidthb").style.width = (100 - x) + 'px';
	}
}

I tried, instead, adding the #176; or whatever the ASCII code was, and it just showed me the ASCII code.  What did I do wrong?  Thanks!

Jim

[seh2000]

I just fund the ° symbol in my word processor symbol library then copied it to:

/usr/local/www/javascript/index/ajax.js

function updateTemp(x) {
if($("tempmeter")) {
$("tempmeter").value = x + '°C';
$("tempwidtha").style.width = x + 'px';
$("tempwidthb").style.width = (100 - x) + 'px';

/usr/local/www/widgets/widgets/system_information.widget.php

Core 1 Temperature

/images/misc/bar_left.gif" height="15" width="4" border="0" align="middle" alt="left bar" />/images/misc/bar_blue.gif" height="15" name="tempwidtha" id="tempwidtha" width="" border="0" align="middle" alt="red bar" />/images/misc/bar_gray.gif" height="15" name="tempwidthb" id="tempwidthb" width="" border="0" align="middle" alt="gray bar" />/images/misc/bar_right.gif" height="15" width="5" border="0" align="middle" alt="right bar" />

Used first the ASCI code, but then the symbol, also I seen some places this around " and other ' around - I use ' hope this helps.
The reason for Core1 instead of CPU is I play with having all four cores displayed.

Steen

[CNLiberal]

Nope, it's still doing the same thing for me.  I guess I'll have to wait until someone develops a system package.  Not a big deal.  I'm just curious why it's not showing a temperature, even though I can query through a sysctl statement and get 4 temperatures (with HyperThreading, two temperatures really).  Thanks all!  I'll still keep an eye on this thread.

jim

[jimp]

You should really use °, don't hardcode such a symbol directly in html/js

[tix]

The Intel 386 coretemp file works for the VIA architecture as well.  I followed the instructions on page 1 of this thread and changed /usr/local/www/includes/functions.inc.php to use "hw.acpi.thermal" instead of "dev.cpu.0.temperature".

Works a champ until an update then I need to reinstall the coretemp.ko module and modify functions.inc.php again.

My version of /usr/local/www/includes/functions.inc.php

function has_temp() {

	/* no known temp monitors available at present */

	/* should only reach here if there is no hardware monitor */
	return true;
}

function get_temp() {
	$temp_out = "";
	exec("/sbin/sysctl hw.acpi.thermal | /usr/bin/awk '{ print $2 }' | /usr/bin/cut -d 'C' -f 1", $dfout);
	$temp_out = trim($dfout[0]);

	return $temp_out;
}

[jimp]

Put the .ko files in /boot/modules and not /boot/kernel, then they won't go away after every update and they still will load as expected.

[stephenw10]

I would think it would be better to have the get_temp function return a value including the ° symbol.
E.g. 45.2°
That way only one file has to be altered.

@jimp:

You should really use °, don't hardcode such a symbol directly in html/js

I agree. I was surprised it worked actually. It probably depends on what your viewing it on.

Steve

[stephenw10]

Using something like:

function get_temp() {
	$temp_out = "";
	exec("/usr/local/bin/mbmon -I -c1 -T3", $dfout);
	$temp_out = (trim($dfout[0]).'°');

	return $temp_out;
}

Would be better except that the above code only correctly returns ° when it it called from the system widget php code. When it's called from the javascript update function it just shows °.  :-\

Steve

[jimp]

Needs a semicolon after that, it's "°"

[seh2000]

@stephenw10:

I would think it would be better to have the get_temp function return a value including the ° symbol.
E.g. 45.2°
That way only one file has to be altered.

@jimp:

You should really use °, don't hardcode such a symbol directly in html/js

I agree. I was surprised it worked actually. I probably depends on what your viewing it on.

Steve

Steve and jimp,
Sorry for been slow, but what you say is I should not really add the ° symbol to the ajax.js file? But could I add the ASCII code there or better not at all? As I mentioned I am not a good programmer :(
Steen

[CNLiberal]

Yeah, this still isn't working for me.  I'm just going to wait until a package is released.

[jimp]

It's bad form (invalid html/js/etc) to put in a symbol instead of the html entity. It may work, but it's not guaranteed to do so.

http://www.w3schools.com/tags/ref_entities.asp