1132 replies to this topic

[Meega]

@wanwizard

 

De eerste hoort vandaag te zijn!

En dan de rest van de uren vandaag.

 

 weer1.jpg   187.53KB   1 downloads

[WanWizard]

Je ziet dus dat dit bij mij niet het geval is, daar staat toch echt "Sa" van zaterdag... Terwijl het hier echt ook nog steeds vrijdag is.

[Meega]

Als ik bij Oss kijk.

 

 oss.jpg   185.07KB   3 downloads

 

 

[WanWizard]

Precies, zo zou het moeten zijn, maar zo is het niet.

 

Ik vermoed dat er wat te veel assumpties in de code worden gemaakt, bijvoorbeeld over timezone, en dat er daar iets mis gaat.

 

Edit: voor de duidelijkheid, ik woon dus niet in Oss, maar dat was de kortste naam die ik kon bedenken om even een test locatie in te voeren.

[Meega]

Precies, zo zou het moeten zijn, maar zo is het niet.

 

Ik vermoed dat er wat te veel assumpties in de code worden gemaakt, bijvoorbeeld over timezone, en dat er daar iets mis gaat.

 

Edit: voor de duidelijkheid, ik woon dus niet in Oss, maar dat was de kortste naam die ik kon bedenken om even een test locatie in te voeren.

Lol

En ik niet in Aalsmeer alhoewel ik daar wel veel kom in de weekenden. (zomers dan)

[Caught]

In ieder geval als ik jou tijdzone instel dan krijg ik net hetzelfde

Het positieve is, dat het enkel de dagen betreft (opgeschoven met 1 dag) ik bedoel de rest klopt.

Ik kom hier op terug.  

[WanWizard]

Blij dat het te reproduceren valt, dat is de eerste stap naar een oplossing.

 

Goed te weten dat de onderste rij wel klopt, want daar ging het me om.

[Caught]

Precies, zo zou het moeten zijn, maar zo is het niet.

 

Ik vermoed dat er wat te veel assumpties in de code worden gemaakt, bijvoorbeeld over timezone, en dat er daar iets mis gaat.

 

Edit: voor de duidelijkheid, ik woon dus niet in Oss, maar dat was de kortste naam die ik kon bedenken om even een test locatie in te voeren.

 

Wil je even deze versie proberen, voor ik hem op Git plaats?

Ik wil zien of er door de aanpassing geen andere lijken uit de kast rollen

Attached Files

•  plugin.rar   9.65KB   1 downloads

[WanWizard]

Dat lost het probleem voor mij op. Merci voor de snelle response !

 

Ik heb nog wel een opmerking, voor als je nog eens tijd hebt: als ik de info opvraag na 22h (dus als de onderste rij leeg is), dan ontbreekt alle weerinfo van vandaag.

Dus in het bovenste blokje staat dan "n/a" in plaats van de temperatuur, geen windrichting en neerslag info, etc.

 

Geen idee of dit een soortgelijk probleem is?

[Caught]

Geen probleem laat wel wat horen als er wat misgaat met de dagen.

Voor je vraag:

Het is eigenlijk een gemiddelde van wat er beschikbaar is, tot op het ogenblik een gemiddelde niet meer kan berekend worden.

Vandaar dan de ontbrekende info. Schakelen naar bv. de actuele waarde zou redelijk complex zijn. 

Ik zeg niet dat ik het nooit anders zal proberen te doen, maar momenteel heb ik geen plannen hiervoor.

We zien wel hé...

[WanWizard]

Ah, dus de bron levert geen actuele weergegevens? Hmm....

 

In ieder geval goed te weten dat dat bovenste blokje dus geen enkel stukje zinvolle informatie levert.

[Caught]

Het brengt me wel op het idee om de "bovenste blok" alsnog van actuele gegevens te voorzien.

Dit betekent dan wel dat de volgende dagen steeds dezelfde info zal tonen "in het bovenste blok" i.p.v. een forecast daggemiddelde.

Ik ga er toch eens werk van maken. 

[Caught]

Zo het viel eigenlijk nog mee om de aanpassingen te doen.

Ik doe nog geen upload naar Github omdat ik nog wil zien of deze versie vannacht geen chrash maakt.

Wel een plugin.py in bijlage voor de mensen die willen testen.

Het wordt dus meer logisch zoals de screenshot laat zien. "idee aangebracht door WanWizard" 

 

 

 

 

 

Attached Files

•  plugin.py   67.03KB   5 downloads

[WanWizard]

Ik heb nog een heleboel vragen hoor...

 

Bijvoorbeeld, de tijden op de onderste rij houden geen rekening met de timezone. Dus ik zie vandaag 18h en 21h voor mijn woonplaats, terwijl ik voor "Oss" 19h en 22h zie.

 

Terwijl jij dit laat bezinken zal in de plugin.py weer even testen.

 

edit: oh, wacht ff, jij haalt de data van een nederlandse site af, dus die 19h uur is dus CET. Snap 'm al.

Edited by WanWizard, 7 January 2017 - 17:57.

[WanWizard]

Ik zie geen problemen met deze plugin.py.

[Pedro_Newbie]

De plugin ziet er goed uit alleen op de getoonde data heb ik mijn twijfels, hierbij een screenshot met daarin de data zoals op mijn iPhone uit de plugin Het Weer (what's in a name)


De gegevens uit de iPhone lijken mij beter te kloppen (ook vergeleken met andere app's)

Let ook op de windrichting

Edited by Pedro_Newbie, 7 January 2017 - 19:40.

[Caught]

Pedro, Ik denk dat je een andere Haarlem vast hebt.

Plaats er eens /nl achter

[Caught]

De andere Haarlems zal ik verwijderen uit de database!

Maar de windrichting zal ik idd nog eens nakijken.

[Pedro_Newbie]

Beide Haarlems geven hetzelfde aan gegevens, volgens jouw plugin is het nu 7 gr. volgens de iPhone 1,6
Ook zouden er hevige buine zijn in Haarlem maar dat klopt ook niet.
Ik zal er morgen eens verder naar kijken

[Caught]

Ik zie het al, op http://www.buienrada...2755003/5daagse

Staat er vandaag geen info meer voor Haarlem (komt uit de 5-daagse) waardoor de fouten voorkomen.

Ik had het nog niet eerder gezien, hopelijk komt het niet te vaak voor.