59_Weather.pm - Vorschläge

betateilchen · 12 Januar 2019, 20:34:24

Zuerst: Danke für die Überarbeitung an alle Beteiligten.

Heute habe ich umgestellt auf das neue Modul + DarkSkyApi, funktioniert soweit auch ganz gut. Zwei Dinge hätte ich für die Wunschliste.

Luftdruckangabe in hPa fände ich zeitgemäßer als mbar (im Reading state)
ein Attribut "expert" (analog zu HomeMatic) mit dem man den Umfang der dargestellten Readings einstellen kann. In manchen Fällen brauche ich gar keine forecast, in anderen Fällen nur die nächsten zwei Tage. Gab es nicht früher ein solches Attribut in dem Modul?

satprofi · 12 Januar 2019, 20:35:30

wie umgestellt?
dachte erst morgen?

marvin78 · 12 Januar 2019, 20:36:57

Per FHEM Update ab morgen. Im SVN wird es schon liegen.

CoolTux · 12 Januar 2019, 20:47:32

Hallo Udo,

Ein expert Mode finde ich gut. Das deaktivieren des Forcast ist ohne weiteres Möglich, wenn man den Code entsprechend an passt.
Ein beschränken des Forcast nur bedingt. Der Forcast kann Stunden sein (wie bei OpenWeatherMap) oder eben Tage. Müsste man mal schauen. Im Grunde würde man nur sagen wie viel Forecast Datensätze geschrieben werden sollen. Egal ob Tage oder Stunden gemeint sind.
Das von Dir gemeinte Attribut ist bestimmt das welches Du beim erstellen des Weblinks als 2 Parameter an die Funktion mit übergibst.

Grüße

betateilchen · 12 Januar 2019, 20:52:35

Ja, kann sein, dass ich das mit dem weblink verwechsle.

Ob forecast sich auf Stunden oder Tage bezieht, ist mir eigentlich wurscht. Wenn ich expert = 3 setze, möchte ich gern drei Datensätze haben. Wenn ich expert = 0 setze, gar keine. Wenn das Attribut fehlt = alle anzeigen.

Zwei Dinge sind mir in der commandref aufgefallen

bei den readings steht .locense statt .license
die Tabellen zu den API sind sehr unübersichtlich, insbesondere weil bei openweathermap plötzlich darkskyapi drinsteht

CoolTux · 12 Januar 2019, 20:57:36

Zitat von: betateilchen am 12 Januar 2019, 20:52:35
Ja, kann sein, dass ich das mit dem weblink verwechsle.

Ob forecast sich auf Stunden oder Tage bezieht, ist mir eigentlich wurscht. Wenn ich expert = 3 setze, möchte ich gern drei Datensätze haben. Wenn ich expert = 0 setze, gar keine. Wenn das Attribut fehlt = alle anzeigen.

Zwei Dinge sind mir in der commandref aufgefallen

bei den readings steht .locense statt .license
die Tabellen zu den API sind sehr unübersichtlich, insbesondere weil bei openweathermap plötzlich darkskyapi drinsteht

Vielen Dank für die Bug Meldung.
Bei Deinem Beispiel würde ich nicht auf das Attribut expert kommen sondern eher als Attribut forecast nehmen.
forecast 3
Drei Datensätze für Forecast. Bei Null keine und wenn kein Attribut alle.

Was hälst davon? Oder verstehe ich Dich da nicht richtig?

Grüße

Benni · 12 Januar 2019, 21:00:31

Vielleicht wäre es besser, das Attribut eher forecast oder forecastCount oder noch besser forecastLimit zu nennen, statt expert. Es handelt sich ja nicht um einen Expertenmodus im eigentlichen Sinn.

gb#

Edit: 2 Dumme ein Gedanke

CoolTux · 12 Januar 2019, 21:06:07

forecastLimit gefällt mir!

betateilchen · 12 Januar 2019, 21:07:56

nenne es, wie Du willst

Es geht wieder um das unlösbare Thema "Standardisierung" in FHEM, darüber will ich nicht schon wieder diskutieren.

CoolTux · 12 Januar 2019, 21:17:58

Zitat von: betateilchen am 12 Januar 2019, 21:07:56
nenne es, wie Du willst

Es geht wieder um das unlösbare Thema "Standardisierung" in FHEM, darüber will ich nicht schon wieder diskutieren.

Gerade bei Standardisierung bin ich voll bei Dir.
Expert ist schön und gut, sagt aber erstmal nur aus das man mehr Möglichkeiten hat. Entweder mehr setter oder mehr getter, oder mehr Attribute die Sichtbar werden. So wäre zu mindest meine Auffassung.

Ich werde morgen mal was vorbereiten mit forecastLimit und es in mein Git schupsen. Würde mich freuen wenn Du es dann testen magst. Ich gebe Bescheid wenn es soweit ist.

KernSani · 12 Januar 2019, 21:36:54

Tschuldigung, wenn ich da kurz einhake (bitte ignorieren wenn ich falsch liege) - ich bin mir nicht ganz sicher, ob das selbe Verständnis bezüglich forecastLimit besteht
* Udo hätte gerne die Möglichkeit "den Umfang der dargestellten Readings " einzustellen - bei gleichbleibender Funktionalität (kompletter Forecast wird geladen)
* Unter forecastLimit würde ich verstehen, dass nur n Datensätze geladen werden (Proplanta kann das)

Ich fände beide Funktionalitäten gut, wobei ich - ehrlich gesagt - sehr selten in das Modul reinschaue, außer ich will irgendwas basteln - meistens bekomme ich alles was ich häufig brauche über readingsgroup/weblink...

CoolTux · 12 Januar 2019, 21:49:49

Zitat von: KernSani am 12 Januar 2019, 21:36:54
Tschuldigung, wenn ich da kurz einhake (bitte ignorieren wenn ich falsch liege) - ich bin mir nicht ganz sicher, ob das selbe Verständnis bezüglich forecastLimit besteht
* Udo hätte gerne die Möglichkeit "den Umfang der dargestellten Readings " einzustellen - bei gleichbleibender Funktionalität (kompletter Forecast wird geladen)
* Unter forecastLimit würde ich verstehen, dass nur n Datensätze geladen werden (Proplanta kann das)

Ich fände beide Funktionalitäten gut, wobei ich - ehrlich gesagt - sehr selten in das Modul reinschaue, außer ich will irgendwas basteln - meistens bekomme ich alles was ich häufig brauche über readingsgroup/weblink...

Da stehe ich gerade auf dem Schlauch.
Wo ist da jetzt der Unterschied? Aber um es mal kurz zu sagen. Es werden immer alle Forecastdaten vom Anbieter geladen und auch im erstellten API Object als Cache vorgehalten. Nur das scheiben der Readings wird begrenzt auf die Anzahl welche angegeben ist. So wäre meine Idee.
Bei den jetzigen API's ginge das noch nicht mal das nur n Forcast Datensätze geladen werden.

LuckyDay · 12 Januar 2019, 21:50:29

Proplantana

KernSani · 12 Januar 2019, 22:02:42

Zitat von: CoolTux am 12 Januar 2019, 21:49:49
Da stehe ich gerade auf dem Schlauch.
Wo ist da jetzt der Unterschied? Aber um es mal kurz zu sagen. Es werden immer alle Forecastdaten vom Anbieter geladen und auch im erstellten API Object als Cache vorgehalten. Nur das scheiben der Readings wird begrenzt auf die Anzahl welche angegeben ist. So wäre meine Idee.
Bei den jetzigen API's ginge das noch nicht mal das nur n Forcast Datensätze geladen werden.

Ok, dann bitte ignorieren

Ich habe dein Forecast-Limit so verstanden, wie das bei Proplanta der Fall ist - dass einfach weniger Daten geladen werden. Einen "expert" level dagegen so, dass nicht nur die Anzahl der angezeigten FC-days, sondern möglicherweise auch die Art der readings in der Anzeige eingeschränkt wird.

betateilchen · 12 Januar 2019, 22:03:37

Mir geht es darum, dass nur eine bestimmte Anzahl von readings aus den vom Wetterdienst geladenen Daten erzeugt wird.

Welchen Sinn es hat, den gesamten Datenbestand im Cache vorzuhalten, verstehe ich nicht. Muss ich aber auch nicht.