MMM-NOAA - Another Weather Module

[redink]

I know you are working on V3, and I am looking forward to it. But, I’ve been having issue with V2. Specifically, it wouldn’t work with MMM-CalendarExt. Meaning I couldn’t have both on screen at the same time.

I looked at the errors, and the issue appears to be with the following lines of code

 	if (this.config.units != "metric") {
	 tt.innerHTML = n Day ~ " + allDay[0].fcttext + "<br />" 
	 	: "Night ~ " + allDay1 + "<br />";
	 	} else {
	 tt.innerHTML = n " + allDay[0].fcttext_metric + "<br />" 
	 	: "" + allDayM1 + "<br />";  
	 	}

The error was something about an uncaught exception from a promise. My JavaScript is very rusty to actually try and fix this on my own, but simply commenting out those lines actually made the error go away, and I was able to get NOAA and CalendarExt on screen at the same time.

Just posting this because I spent the last two days trying to get this resolved, and figured someone else might benefit.

[cowboysdude]

@solelo said in MMM-NOAA - Another Weather Module:

@mykle1 I can tell there is a lot of love in that “maniac”, he is just trying to tease us.

[SoleLo]

@cowboysdude can we have the V3… you killing us…

[cowboysdude]

@solelo Working out some things still ;) Hang in there

[Mykle1]

@cowboysdude said in MMM-NOAA - Another Weather Module:

Working out some things still Hang in there

We want the world and we want it now. :-)

[justjim1220]

@cowboysdude

Yeah, what Mykle1 said… Now dammit! :face_with_stuck-out_tongue_closed_eyes:

[cowboysdude]

John Isabella: I’m truly sorry about your brother. It’s too bad.

Truman Gates: Oh no sir. You ain’t seen bad yet, but its coming.

:face_with_tears_of_joy:

[cowboysdude]

@redink Well 3 is close…so if you have issue with that PLEASE let me know… It should be by this weekend…

[cowboysdude]

Just an FYI here’s the README for NOAA 3…

Testing a few things but should be done by this weekend :)

Enjoy reading! Hope you love it as much as I loved working on it!

https://github.com/cowboysdude/MMM-NOAA3/blob/master/README.md

Here’s a little preview…

[cowboysdude]

Description:

NOAA3 V3 Lots of changes. IF you have any issues please report here :)

Cannot say this enough…PLEASE read the README. This was basically rebuilt from the ground up and has MANY changes.

Enjoy

MMM-NOAA3 V3.00
Please follow Readme on Github!

Screenshots:

Version 3.0.0

Download:

[card:cowboysdude/MMM-NOAA3]

[Mykle1]

@cowboysdude

I really wish I could say something bad about this module. ;-)

With that, congrats on a slick piece of work. :-)

[justjim1220]

@mykle1

Amen to that, brother! :smiling_face_with_sunglasses:

[Mykle1]

I’m a little late to the update game but I did want to make sure that there were no issues with the new MM v2.4.1 and NOAA3, and of course, Hello-Lucy. So, I’m happy to report that my favorite weather module (NOAA3) and Hello-Lucy and MM v2.4.1 all work very happily together.

https://youtu.be/MMKvbuGcqpk

[fox]

thank you @cowboysdude .
My MM is in german, but how can i force to show the english names in the NOAA3 module?

[noimagination]

Is there anyway to easily override the latitude/longitude information? I live in the country so the lat/lon returned by http://ip-api.com/json is not very accurate for my location. Also, what other options are available for configuring NOAA3 in config.js?

[justjim1220]

@fox

try adding the following in the config.js

langTrans: "en",

example…

{
    module: 'MMM-NOAA',
    config: {
        langTrans: "en",
	apiKey: "YOUR API KEY",
	airKey: "YOUR API KEY"
}
},

Not really sure it will work, but I will try it on mine in a little while and let you know if it does.
If you try it before I get back on here, let me know if it did work or not.

[justjim1220]

@fox

I’m assuming with the upvote that it worked for you?

[cowboysdude]

@justjim1220 No sir that will not work

[cowboysdude]

@fox what do you mean English names? The headings for each like Humidity and such?

[fox]

@justjim1220 said in MMM-NOAA - Another Weather Module:

langTrans: “en”,

thank you for the anser, i tried it but did not work.