How to: Get CPU temp

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

stephenw10

Nope. ???
Same as before, using:


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

	return $temp_out;
}

Results in initially correct ° then 46.5°C
Hmm.

Steve

jimp

It may be getting run through htmlentities somewhere in there then. Might be safe to leave the symbol in there if that is the case.

stephenw10

Unfortunately using:


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

	return $temp_out;
}

Results in the attached image.

Perhaps the character set is being restricted somewhere?

tempwidget.jpg_thumb

jimp

That's a unicode degree symbol, try a standard ascii one, º

seh2000

@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

OK, got you, yes correct…
Cheers Steen

stephenw10

No different. I entered both º and ° and both result in ��. :-\

Steve

jimp

I'd just leave it out.

(Or convert to Kelvin, no symbol there! ;-)

stephenw10

Kelvin nice! :D

I'll stick with the nasty ° addition to ajax.js. I'm clueless about js, how would one enter that as an ascii code instead? Using ° is not working.

Steve

seh2000

@jimp:

I'd just leave it out.

(Or convert to Kelvin, no symbol there! ;-)

Kelvin - LoL can core temp convert the output?

stramato

This is awesome. My i3-540 reads 29C, 29C, 36C, 36C lol.

Will this be included in future snapshots? Or should we go ahead and modify it ourselves?

seh2000

@stramato:

This is awesome. My i3-540 reads 29C, 29C, 36C, 36C lol.

Will this be included in future snapshots? Or should we go ahead and modify it ourselves?

That is cool, which type of case/box you use? I get 54C with an Atom D510 that sits in a Jetway Motherboard that is in a passive cooled M-350 box.
Cheers Steen

asterix

Though this is 2.0 RC feedback section, but since the topic is on coretemp, I wanted to see if there is a pfsense 1.2.3 compatible 32-bit coretemp file out there that I can use. Moved to 1.2.3 last night as 64-bit snapshot snort is broken since weeks and I don't have the patience to wait any longer.

RedRep

@asterix:

Though this is 2.0 RC feedback section, but since the topic is on coretemp, I wanted to see if there is a pfsense 1.2.3 compatible 32-bit coretemp file out there that I can use. Moved to 1.2.3 last night as 64-bit snapshot snort is broken since weeks and I don't have the patience to wait any longer.

Pull the file from a FreeBSD 7.2 install CD.

asterix

I am not a FreeBSD user.