MMM-forecast-io -- Localized up to the minute weather

[pyrosmiley]

@mongo116 sadly I don’t see that those were ever added in and it wasn’t me that did it. I have no idea what I’m doing but sometime soon I’ll see if I can figure it out? I also tried for that awhile back using CSS but didn’t get very far. If you get it let me know, and I’ll do the same!

[morozgrafix]

I’ve added some of those on my local branch a long time ago. Feel free to pick from there https://github.com/morozgrafix/MMM-forecast-io/commit/972201f64d432c3c0863a7d76364e7f8af1bb817 and https://github.com/morozgrafix/MMM-forecast-io/commit/13ff8b78c86dc0905e46c11702d7173571517e79

My fork is fairly old and hasn’t been synced in a while, though.

It looks like this:

[pyrosmiley]

Very nice! I’ll definitely pull from that then

[rak]

Hi guys,

I like the concept of the bars for the min/max temperature very much. I wanted the look of the hourly temperature graph a bit more more modern and slick.

I sat down today and did some code magic. I am surprised how it went. The magicmirror is the first javescript project. So bear with me.

This is how it looks now.

I created a fork which I will clean up a bit in the next weeks. Today I am glad it works at all ,-)?

Enjoy.

Ralf

[grantc66]

@rak There seems to be a small error in the script creating the graph:

// Init chart.js
this.chart = new Chart(ctx, {
  type: 'line',
  data: data,
  options: options
});

Throws an uncaught reference error. Unfortunately I know absolutely nothing about js, can you let me know how to fix it.

Ta

[rak]

You had the plugin before?

Have you done this after switching to my fork? It should install chart.js as a dependency.

npm install

I hope this solves your problem. Let me know.

[idoodler]

@paulocarrasco Thats an easy one:) You are missing an , after 'my api key'.

Here is the correct config:

{
      module: 'MMM-forecast-io',
      position: 'top_right',  // This can be any of the regions.
      config: {
        // See 'Configuration options' for more information.
        apiKey: 'my api key', // Dark Sky API key.
        // Only required if geolocation doesn't work:
        latitude:   37.1468,
        longitude: -8.5846
      }
},

[grantc66]

@rak Had the original working OK, tried to npm install chart.js in the MM root :(

the npm install in the MMM-foecast.io directory fixed it. :)

looks great, good work.

thanks

[barnosch]

@rak
Nice work. i just installed it and adjusted for fitting my mirror.

Is it possible to shrink the height from the graph?
or is it calculated from the width (400px) to keep its aspect ratio?

[rak]

Should be easy do adapt. I can check if the height is currently fixed or exposed in the config.js.

[barnosch]

guess i found it in the

renderPrecipitationGraph: function ()

and

renderSVGPrecipitationGraph: function ()

both heights are calculated there.

var width = this.config.precipitationGraphWidth; 
var height = Math.round(width * 0.3); // 120 by default

Played around with the setting, but it did not change anything. Don’t get it currently… mabye you have an idea?

[barnosch]

seems not the correct place.
Even if comment out both lines with “height” it makes no difference…

[dudej11]

Hey! Although this thread is a bit old, I’m wondering if I can get some assistance. I have my DarkSky API key correct, however the console of my Mirror is telling me that it refreshed the access token because it had expired. I checked the API panel, and it did indeed show that the API had been called, so my API key is correct. However, the weather module never loaded. Any suggestions?

[barnosch]

Other moduls are showing?
Have you tried with the above example from idoodler?

Otherwise try to delete the folder again, pull it again and don’t forget to “npm install” in the just downloaded folder.

[twosquirrels]

@rak

Thanks for this fork! I like your sample screenshot. I cloned it and did npm install, but the sizing of all the colored elements are all way off. Any ideas?

Why is it showing 10 degrees all the way up through 70 when it really only needs to show 70 to 95?

The rain bars are too tiny, as are the temperature range bars. :disappointed_face:

[rak]

Hello,

because it meant to be for Grad Celcius and not Grad Fahrenheit. When ever will US go to a metric system ,-).

Fix is easy.

Search in modules/MMM-forecast-io/MMM-forecast-io.js for

id: 'temperature',
            position: 'left',
            ticks: {
              suggestedMin: 10,

and change suggestedMin to you choosing, like 70 … .

Enjoy.

For the bars. You need more space. Make the module wider, or reduce the font size (custom.css).

[justindp2006]

Been trying to use this module but the width is taking up almost the whole screen. I’m assuming size needs adjusted in the css file but I don’t know which is causing this. Ideas?

[barnosch]

you can maybe try the following.
Add to your custom.css

.MMM-forecast-io{
transform: scale(0.90, 0.90);
}

Code was not tested. the 0.90 stands for 90%
Maybe that works and is good enough for your needs

otherwise look for the css file within the module folder and play with the settings there

[justindp2006]

Thank you! So interesting tid-bit. It works on Browser View mode (IP:8080) but on the mirror itself, no changes. I also have trouble with another module. It shows on the Browser, but not at the mirror. It must be something with the Pi Zero image I’m using.

[lavolp3]

Guys, going through the thread, I see that a lot of good ideas were implemented in forks but not sent as PRs. I think it would be a good idea to have some of these included into the original. dmcinnes/MMM-forecast-io, since this is the one referenced in the original MagicMirror.

@dmcinnes what do you think?
Do you want this rep to be bloated up or do you want it to be kept mostly as it is and leave major changes to forks?

I would expect at least to have some of @morozgrafix very good ideas implemented as optional (sunset, precipitation probability).
Also @rak 's ideas look very very promising but I admit might be a bit much as changes for the master.