MMM-bergfex not working

MajorC

And it is working.

This is important:

This is an array with strings of the skiareas to display. String must exactly match the name on http://www.bergfex.at/oesterreich/schneewerte/

sdetweil

@MajorC yes, precise config is required in each module. some don’t document well…

MajorC

Thanks for your very fast help. Maybe someone will rewrite this MMM. Winter is coming!

Suboshi

Hi ,

sorry for bringing this up again. I also have the bergfex “loading…” issue. However, in the logs I can see the values are retrieved. They just don’t show in the mirror:

0|mm     | 83 snow reports from bergfex.at retrieved.
0|mm     | [2020-11-12 21:40:29.367] [LOG]    searching for Koralpe
0|mm     | [2020-11-12 21:40:29.369] [LOG]    searching for Hintertuxer Gletscher / Hintertux
0|mm     | [2020-11-12 21:40:29.370] [LOG]    searching for Ischgl
0|mm     | [2020-11-12 21:40:29.373] [LOG]    searching for Obergurgl - Hochgurgl
0|mm     | [2020-11-12 21:40:29.375] [LOG]    searching for Seefeld/Rosshütte
0|mm     | [2020-11-12 21:40:29.376] [LOG]    searching for Sölden
0|mm     | [2020-11-12 21:40:29.377] [LOG]    searching for Stubaier Gletscher / Stubaital
0|mm     | [2020-11-12 21:40:29.377] [LOG]    
0|mm     | [
0|mm     |   null,
0|mm     |   {
0|mm     |     skiarea: 'Hintertuxer Gletscher / Hintertux',
0|mm     |     tal: '-',
0|mm     |     berg: '135 cm',
0|mm     |     neu: '-',
0|mm     |     lifte: '0',
0|mm     |     update: ''
0|mm     |   },
0|mm     |   {
0|mm     |     skiarea: 'Ischgl',
0|mm     |     tal: '-',
0|mm     |     berg: '20 cm',
0|mm     |     neu: '-',
0|mm     |     lifte: '0',
0|mm     |     update: ''
0|mm     |   },
0|mm     |   {
0|mm     |     skiarea: 'Obergurgl - Hochgurgl',
0|mm     |     tal: '-',
0|mm     |     berg: '75 cm',
0|mm     |     neu: '-',
0|mm     |     lifte: '0',
0|mm     |     update: ''
0|mm     |   },
0|mm     |   null,
0|mm     |   {
0|mm     |     skiarea: 'Sölden',
0|mm     |     tal: '19 cm',
0|mm     |     berg: '72 cm',
0|mm     |     neu: '-',
0|mm     |     lifte: '0',
0|mm     |     update: ''
0|mm     |   },
0|mm     |   {
0|mm     |     skiarea: 'Stubaier Gletscher / Stubaital',
0|mm     |     tal: '35 cm',
0|mm     |     berg: '110 cm',
0|mm     |     neu: '-',
0|mm     |     lifte: '0',
0|mm     |     update: ''
0|mm     |   }
0|mm     | ]

So the names of the resorts seem to be correct.
Any ideas on this one?

thanks
Chris

Suboshi

Argh… just 2 minutes after posting I found the issue:

I had 2 values “Koralpe” and “Seefeld/Rosshütte”, which were displayed in the log as NULL. Removing these entries made bergfex work…

Again, it was a typo ;)

thanks
Chris

FatTony123

Hi all,
I’ve created a fork and did some updates to the code and config.
You can now select the country where bergfex should choose the values from.

Feel free to test and let me know any feedback.
So far I can show multiple countries and their values…

MajorC

@FatTony123 can you explain how to use the fork? I am a total noob.

FatTony123

@MajorC Same approach as with other modules.
Get the repository via

git clone https://github.com/FatTony123/MMM-bergfex.git

and then run

npm install cheerio

then

npm install

That installs all relevant software. After that you need to setup your config file accordingly.

I’m not that active here, so you can reach me via github too …

sdetweil

@FatTony123 u shouldn’t have to do the separate install of cheerio as it is in the package.json

FatTony123

@sdetweil Thanks for the hint! (still not that familiar with npm ;-))
Updated the install instructions and package.json file accordingly.

MajorC

@FatTony123 said in MMM-bergfex not working:

Hi all,
I’ve created a fork and did some updates to the code and config.
You can now select the country where bergfex should choose the values from.

Feel free to test and let me know any feedback.
So far I can show multiple countries and their values…

Works like a charm. Thank you very much!

FatTony123

Great to hear, thanks!

Let me know if you have any question, comments or ideas for improvements…

Cheers!

MajorC

I never saw values for “neu” maybe they are not provided by the city. Perhaps this would be nice to have this optional to save some unused space.

And I will play around with the CSS at the weekend. I would like to have some more space between each column.

FatTony123

@MajorC said in MMM-bergfex not working:

I never saw values for “neu” maybe they are not provided by the city. Perhaps this would be nice to have this optional to save some unused space.

Any particular values you are missing / not seeing? The values should get updated once they are updated on bergfex.
Which cities / config are you using? Then I can check on my mirror…

MajorC

@FatTony123 my mistake. It looks like the cities I have in my configurations are not reporting “new snow” (Neu") to bergfex. So I never saw something where. I just added another city and it works quite nice.

As I am not interested in “new snow”, it would be nice to be able to activate/deactivate all (Tal, Berg, Neu, Lifte) as an option in the config. But I can do that by myself in the source code.

FatTony123

@MajorC Sounds like a good idea to set the options that should be visible on the mirror. Let me know once you have something so I can play with it too ;-)

MajorC

I played a little bit around with coustom.css and at the moment it looks nice to me with:

/**
 * MMM-bergfex
 */
.MMM-bergfex td {
	padding-right: 20px;
}

FatTony123

@majorc Thank you, I’ve added that and updated the repo.

MajorC

@fattony123

I added a second element, so it looks nice, even if there is now data available e.g. “new snow”.

.MMM-bergfex td {
	padding-right: 20px;
	min-width: 35px;
}

FatTony123

@majorc thanks, added!