FHEM Forum

FHEM - Entwicklung => FHEM Development => Thema gestartet von: Dirk am 22 Juli 2013, 23:54:30

Titel: Mini-Patch: global attr altitute
Beitrag von: Dirk am 22 Juli 2013, 23:54:30
Hallo Rudi,

anbei ein Mini-Patch für global attr altitute.

Einmal als Anregung von betateilchen von hier (http://forum.fhem.de/index.php?topic=13722.0)

Und ich würde das gerne im Modul i2cBMP180 (//forum.fhem.de/index.php?t=msg&goto=86969&rid=42#msg_86969) benutzen.
Man könnte hier zwar auch ein lokales Attribut benutzen, allerdings fehlt im FHEM derzeit tatsächlich die dritte Dimmension :)

Gruß
Dirk
Titel: Aw: Mini-Patch: global attr altitute
Beitrag von: justme1968 am 23 Juli 2013, 00:01:42
finde ich gut.

im panstamp luftdrucksensor könnte man es auch verwenden.

gruss
  andre
Titel: Aw: Mini-Patch: global attr altitute
Beitrag von: rudolfkoenig am 23 Juli 2013, 08:15:13
Habs eingebaut (+ mini-Doku), obwohl es mir nicht wirklich passt.

Eigentlich muesste aus SUNRISE_EL ein GEO Modul werden (genauso wie Twilight), den man mit long/lat/alt definiert, und der sunrise/etc zur Verfuegung stellt. Andere Module muessten sich per attr mit dem passenden Instanz verbinden.

Wenn ich wuesste, wie man einen schmerzfreien Uebergang hinkriegt, wuerde ich damit sogar anfangen :)
Titel: Aw: Mini-Patch: global attr altitute
Beitrag von: Dirk am 23 Juli 2013, 09:14:41
Super.
Vielen Dank.
Titel: Aw: Mini-Patch: global attr altitute
Beitrag von: Dr. Boris Neubert am 23 Juli 2013, 21:35:49
Zitat von: rudolfkoenig schrieb am Di, 23 Juli 2013 08:15Wenn ich wuesste, wie man einen schmerzfreien Uebergang hinkriegt, wuerde ich damit sogar anfangen :)

Schmerzfrei heißt, daß die Anwender mit minimalen/ohne Aufwand sunset() durch ReadingsVal("myGeo","sunset") ersetzen?

Was spricht dagegen, GEO zunächst einzuführen mit einem Hinweis beim Update, daß SUNSET_EL ein Auslaufmodell ist, und es dann irgendwann später nach erneutem Hinweis beim Update zu entfernen? Die Hinweise beim Update muß jeder Anwender quittieren, sonst gibt es kein Update. Soll später ja keiner sagen, er hätte nichts von alledem gewußt ;-)

Viele Grüße
Boris
Titel: Aw: Mini-Patch: global attr altitute
Beitrag von: Dirk am 24 Juli 2013, 08:46:40
Hi Rudi,

danke übrigens für das korrigieren des Typo in Altitude.
Allerdings fehlt nach altitude jetzt ein Leerzeichen.

Gruß
Dirk
Titel: Aw: Mini-Patch: global attr altitute
Beitrag von: rudolfkoenig am 24 Juli 2013, 08:52:10
Habs gefixed, getestet, eingecheckt und fuer update zur Verfuegung gestellt.