Temp suddenly showing 239 degrees

[KirAsh4]

No matter what I do, I can’t seem to reproduce this odd temperature display. I’ve even started from scratch with a different rpi, completely blank to having MM up and running … no odd temperature.

[weezer05]

Hi there,

I experienced this the other day, and I checked again - my new pi 3 is still showing an incorrect temperature reading as well. It seems as though the temperature is set to ‘Kelvin’, though I set the temperature to ‘imperial’, within the config.js configuration file.

---------------------------------------------------------
/* Magic Mirror Config Sample
 *
 * By Michael Teeuw http://michaelteeuw.nl
 * MIT Licensed.
 */

var config = {
        port: 8080,

        language: 'en',
        timeFormat: 12,
        units: 'imperial',

---

Note from admin: Please use Markdown on code snippets for easier reading!

[weezer05]

After looking around, I see that github member p0akley seems to have identified an issue 5 days ago that matches this description. Following his instructions I was able to resolve the issue with my Pi 3.

Seems there is a small adjustment needed in the file:
modules/default/currentweather/currentweather.js

Link to explanation / resolution:
https://github.com/MichMich/MagicMirror/commit/25ddf2c65178e2b8afe0709b48d5d30054208c04

[KirAsh4]

That’s been in the 'develop' branch. Not pushed to 'master' yet. With this being a “biggie” for everyone who’s running the master branch, perhaps @MichMich might consider pushing it to the master branch sooner rather than later.

And my apologies earlier for not having spotted it. I always run the develop branch. #livingontheedge :)

[MichMich]

If one of you can confirm the develop branch fixes the issue, I’ll push the changes to the master branch.

[weezer05]

Been trying to reproduce this now for the past hour, and I can’t get it to show up incorrectly again. I double checked line 167 of the currentweather.js file on the develop branch, and it does include the correction mentioned earlier which initially resolved my issue.

[jtmoore81]

@michmich Was this ever pushed? I believe I’m having the same issue.