MMM-aviationwx (Aviation weather)

320fan · Dec 7, 2017, 2:04 PM

As I can see you continue to improve your mode - thats fine, thanks. Now I’ve some time for tests of this updated module. My points are:

No possibility to set custom number of airports for the “Static” view: for example, line like:

ICAO: "UUDD,UUWW,UUEE,UUOO,UWKD,URMT,ULLI,UWWW",  // separated by comma only

works fine (8 ICAO codes), but when I’m decrease/increase number of ICAO codes it doesn’t work: “Loading PilotWX…” and thats all.

My next proposal is to remove line “Station Conditions” for the Static view. It’s a little bit pointless ;)
For the “Rotating” view: the whole text alltime yellow. How I can change that?

3.1) For the “Rotating” view: pressure must be like xx.xx (not like xx as it is). In aircraft you must use all four digits for all the time. But for me - pressure in hectopascals (like 1019, without digits after comma) better in any case. Hovewer, in any case 1 hg = 33.863886666667 hPa. hg used for US airports at all the time, for EU/RU airports - hectopascals are the standard.
3.2) And the last one regarding pressure - it’s nice to have it for a “Standard” view too - it’s a really important information.

About time, which I can observe in the end of the line (tt:tt): I would ask you to display a UTC (Zulu) time, not a local time. So, thats the standard of METAR/TAF ;) In that case you display it like (ttttZ) or (tt:ttZ), and Zulu will be a remider for that.
Thanks for visibility in KM, thats very usefull. And to have a same view - please, change speed in knots as “KT”, not a “kt”.
For the “Loading PilotWX…” message - I would like to propose next code:

        if (!this.loaded) {
            wrapper.classList.add("wrapper");
//            wrapper.innerHTML = "Loading PilotWX . .";
//            wrapper.className = "bright light small";
              wrapper.innerHTML = (this.loaded) ? this.translate("EMPTY") : this.translate("LOADING");
              wrapper.className = "small dimmed";

            return wrapper;
        }

In that case this module will take into account locale settings for correct displaying of “Loading” message.

In case of no cloudbase layer classification (like "BKN/OVC/etc.) you may see a OVX text:

Thats not correct; the reply from server in that case is: sky_condition sky_cover=“OVX” cloud_base_ft_agl=“0”

The right way - to display data directly from a string, as example:
raw_text> URMT 071330Z 32006MPS 0650 0550NE R25/0800N +SHSN VV002 M01/M01 Q1010 R25/290255 NOSIG RMK QBB080 OBST OBSC QFE719 raw_text>

VV002 means vertical visibility in hundred of feets or QBB080 means vertical visibility in meters (this code not applicable for all airports).

And if case of aligment to the right:

               {
                        module: 'MMM-PilotWX',
                        header: 'Aviation weather',
                        position: 'bottom_right',
                        disabled: false,
                        config: {

the whole layout is shifted to the right; Is it possible to align/to stretch all elements like for the “Weather forecast” plugin?

Is it possible to use a same font/font size, like for standart modules (like "calendars, “weather”, etc.) to keep a same view for the whole mirror? It’s not a big deal, but in that case we can make a mirror looks more “arranged”.

And last, but do not least: @Mykle1, thanks again for your job.

320fan · Dec 7, 2017, 4:45 PM

one moew issue:

Datas from: https://aviationweather.gov/adds/dataserver_current/httpparam?dataSource=metars&requestType=retrieve&format=xml&stationString=UUDD&hoursBeforeNow=1

raw_text
UUDD 071630Z 34005MPS 9999 BKN022 M04/M08 Q1009 R88/520295 NOSIG
/raw_text

sky_condition sky_cover="BKN" cloud_base_ft_agl="2200"

It must be BKN022, not BKN2200.

Mykle1 · Dec 8, 2017, 1:59 AM

@320fan said in MMM-aviationwx (Aviation weather):

No possibility to set custom number of airports for the “Static” view

Yes, at the moment you MUST use 8. However, another user has graciously asked if he could modify some things and issue a PR that I will merge into the module. That is one of the changes.

My next proposal is to remove line “Station Conditions” for the Static view.

You mean the headers, “Station” and “Conditions”? Add this to your custom.css file and it will remove both

.MMM-PilotWX .station {
  display: none;
}

And this to change to any color for rotating

.MMM-PilotWX .bottom_bar {
  color: yellow;
}

pressure must be like xx.xx (not like xx as it is)

Tell that to the API. That’s the data being given. I just display it.

Is it possible to use a same font/font size, like for standart modules (like "calendars, “weather”, etc.

Use css to customize to your liking

Alignment will always be an issue because the module is not set in a table. I did mention above that I am not capable of using tables yet. SO, for the time being, left regions will be better suited to the module

Taking specific portions of the raw_text for display is beyond me.