Can anyone revive AviationWX, PilotWX, or TAF

[sdetweil]

@FSAHD is the problem just the use of request?
that can be fixed w very little code change

[FSAHD]

@sdetweil that is what I believe it it, also what the AI helped me deduce.

I attempted to solve that but was unsuccessful. I believe that was the problem with all 3 modules I mentioned.

I tried using npm install request. that didnt work.

Then I attempted to update the code to use axios

[sdetweil]

@FSAHD OK, let me look at them

[sdetweil]

@FSAHD well every url changed, and all the data
changed. some has to be calculated (avg airport delay) or that info display deleted.

too much work for me., given my avaliable time

[FSAHD]

@sdetweil I appreciate you looking into this. ill take some time and see wat I can do!

[sdetweil]

@FSAHD I got the metar to work, then the faa data is all different not by airport request. and all the variable names have changed too

don’t know if the data is different for display

and the data is xml

[FSAHD]

@sdetweil I know the airport delay item is last on the wish list.

I can take over and update variables and such. wherever you left off ill be happy to take over!

[sdetweil]

@FSAHD ok, let me finish the faa data . I was working on it this morning

[sdetweil]

@FSAHD ok, here is the partially updated MMM-aviationwx

https://github.com/sdetweil/MMM-aviationwx

there is a file of the output sent back from the node_helper( this morning) in formatted json (sample-data.txt)

so you can see the field names and values…

I marked in the MMM-aviationwx.js in getDom() where all this needs to be corrected with correct variable names and values…

as an example

      var icao = airport.id;

airport.id is now airport.station_id
“station_id”: “KSFO”,

[FSAHD]

@sdetweil Thanks again for the help.

I think I have all the variables updated. Im c urrently working on trying to get the dependencies to install on my pi so I can test.

You were a big help and I see where I went wrong in a lot of places when I was trying to build from scratch. It was a lot of simple mistakes lol

[sdetweil]

@FSAHD great news.

dependencies?

package.json has them all for the module
so npm install will do all for the module

use my install script for mm

[sdetweil]

@FSAHD a lot of simple mistakes…

computers do things fast. whatever we tell them… lol

[FSAHD]

@sdetweil I tried that after I cloned the git into the modules folder.

I ended up getting this error.

I also don’t remember if I used your script for this instance of mm or if I installed manually if that makes any difference

[sdetweil]

@FSAHD ok, let me look at it in a couple hours

[mumblebaj]

@sdetweil I think to resolve just do npm uninstall request and will remove request from the package,json. Else it will try to install it and you will run into the errors above.

[sdetweil]

@mumblebaj but the code uses request…

It downloads a now gz folder, with one xml file in it… I tried to use fetch and couldn’t get it to unzip the file in stream…
2 days, gave up

I installed ok on a pi4 after doing my script install
warnings but no failures

npm WARN deprecated hoek@6.1.3: This module has moved and is now available at @hapi/hoek. Please update your dependencies as this version is no longer maintained an may contain bugs and security issues.
npm WARN deprecated uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm WARN deprecated request@2.88.2: request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated joi@13.7.0: This version has been deprecated in accordance with the hapi support policy (hapi.im/support). Please upgrade to the latest version to get the best features, bug fixes, and security patches. If you are unable to upgrade at this time, paid support is available for older versions (hapi.im/commercial).

added 66 packages, and audited 67 packages in 46s

2 packages are looking for funding
  run `npm fund` for details

6 vulnerabilities (2 moderate, 4 high)

To address issues that do not require attention, run:
  npm audit fix

To address all issues possible (including breaking changes), run:
  npm audit fix --force

Some issues need review, and may require choosing
a different dependency.

Run `npm audit` for details.
sam@raspberrypi4:~/MagicMirror/modules/MMM-aviationwx $ 

[sdetweil]

@FSAHD i just pushed a fix to package.json and a new postinstall script

git pull to get the updates…(watch out that it doesn’t overwrite your code)
then the postinstall script from the module folder

./postinstall

it will be called by npm install as well…

[mumblebaj]

@sdetweil I think it is possible with axios. Let me have a look at it. Will fork module from you and see what I can do and push a PR if it works.

[sdetweil]

@mumblebaj said in Can anyone revive AviationWX, PilotWX, or TAF:

I think it is possible with axios

really don’t want axios… better to get native fetch working…
thats what I was trying. and you can use node xxx.js to test it without MM in the way…

its ‘possible’. just all the different ‘it works like this’ didn’t work.

there are two fetches in the node helper… one gets the gz file
and one gets an xml file… that one should be easy, but I didn’t want to mix in the same code path

[mumblebaj]

@sdetweil The below code works fine. Tested it. Having issues with my system at the moment and can’t make the changes. Can you incorporate?

async function fetchData(metarUrl, airports) {
  try {
    const response = await axios.get(metarUrl, {
      responseType: 'arraybuffer', // ensure response is treated as binary
      headers: {
        'Accept-Encoding': 'gzip', // request gzip compression
      },
    });

    // Unzip the gzip response
    const unzippedData = zlib.gunzipSync(response.data).toString();

    let json_string = convert.toJson(unzippedData);
    var json = JSON.parse(json_string);
    var metarData = json.response.data.METAR;
    var airportData = new Object();

    var airports = payload.airportList.split(",").map(function (airport) {
        airport = airport.trim();
        return (airport.length < 4) ? "K" + airport : airport;
      });

    airports.forEach(function (airport) {
        metarData.forEach(function (metar) {
            if (airport === metar.station_id) {
                if (this.debug)
                    console.log("METAR data found for " + airport);
                airportData[airport] = metar;
                return; // check next airport in list
            }
        });
        console.log(metarData)
    });
  } catch (error) {
    console.error('Error fetching data:', error);
  }
}