neue Features: Attribut myReadings

Begonnen von Damian, 23 April 2017, 17:15:14

Vorheriges Thema - Nächstes Thema

Damian

Zitat von: Fixel2012 am 03 Juli 2017, 10:53:59
hi Damian,

wann ist es geplant diese DOIF-Funktion offiziell zu verteilen?

Danke und Gruß,

Fixel

Vermutlich im Laufe des Monats.
Programmierte FHEM-Module: DOIF-FHEM, DOIF-Perl, DOIF-uiTable, THRESHOLD, FHEM-Befehl: IF

Ellert

Bevor es ein offizielles Release gibt, möchte ich den Attributnamen zur Diskussion stellen.

Ich meine, "my" ist etwas zu nichtssagend, "user" ist schon vergeben und "usor" für usorReadings, ist vieleicht nicht unterschiedlich genug.

Wäre nicht "optReadings" für optionalReadings oder "extReadings" für extendedReadings eine treffendere Bezeichnung?

Damian

Zitat von: Ellert am 09 August 2017, 14:08:51
Bevor es ein offizielles Release gibt, möchte ich den Attributnamen zur Diskussion stellen.

Ich meine, "my" ist etwas zu nichtssagend, "user" ist schon vergeben und "usor" für usorReadings, ist vieleicht nicht unterschiedlich genug.

Wäre nicht "optReadings" für optionalReadings oder "extReadings" für extendedReadings eine treffendere Bezeichnung?

Ich bin da offen, ggf. uReading oder doifReading, weitere Vorschläge kann man hier einbringen und dann schauen wir mal, was für die Anwender verständlich erscheint: für Userreadings mit DOIF Syntax
Programmierte FHEM-Module: DOIF-FHEM, DOIF-Perl, DOIF-uiTable, THRESHOLD, FHEM-Befehl: IF

Fixel2012

Zitat von: Damian am 09 August 2017, 17:52:38
Ich bin da offen, ggf. uReading oder doifReading, weitere Vorschläge kann man hier einbringen und dann schauen wir mal, was für die Anwender verständlich erscheint: für Userreadings mit DOIF Syntax

doifReading erscheint mir persönlich am geeignetsten. Und ist am besten verständlich, was überhaupt gemeint ist!
Fhem 5.8 auf Raspi 3, HMLAN und 868MHz CUL mit einigen Komponenten, Z-Wave Rollladenaktoren, Tablet UI, 433 MHz CUL mit Baumarktsteckdosen und Temp Sensoren, Amazon Echo, Echo Dot, 2x SONOS  play1, 1x SONOS Connect AMP,  presence, HUE, Lightify

Shojo

Zitat von: Fixel2012 am 09 August 2017, 18:11:10
doifReading erscheint mir persönlich am geeignetsten. Und ist am besten verständlich, was überhaupt gemeint ist!

Ja sehe ich genauso :)

Vote doifReading
FHEM auf: Shuttle PC (x64) (Docker)
Bridge: SignalESP 433mHz, ConBee (deCONZ in Docker)
Rest: ESP8266, SONOFF, Sonos, Echo Dot, Xiaomi Vacuum (root), ESP RGBWW Wifi Led Controller, Node-RED, LEDMatrix, Pixel It

Ellert

Ein Attributsname sollte einen Hinweis auf seinen Inhalt geben, da halte ich "doif" als Vorsilbe für nicht geeignet. Diese Vorsilbe könnte dann jedes doifspezifisches Attribut haben.

Die "Userreadings mit DOIF Syntax" werden in der Regel ausgeführt, sie leisten etwas, vollbringen etwas.

Daher wäre "performReadings" ein treffender Attributsname.

Per

Zitat von: Ellert am 09 August 2017, 21:04:24Diese Vorsilbe könnte dann jedes doifspezifisches Attribut haben.
Genau darum ist es ja gut geeignet. Und es wird auch in Zukunft nicht zu namentlichen Doppelungen kommen, da es nur ein Device DOIF gibt (geben kann).

Zitat von: Ellert am 09 August 2017, 21:04:24"Userreadings mit DOIF Syntax"
Ein weiterer Punkt für doifReading.

Zitat von: Ellert am 09 August 2017, 21:04:24werden in der Regel ausgeführt, sie leisten etwas, vollbringen etwas.
Und nocheiner. Zumindest aus Sicht der DOIF-Fanboys* wie mich ;)

*) Bestes Beispiel: OliS. mit "DOIF, ich will ein Kind von dir" ;D.

Damian

#22
Ich denke, dass dieses Attribut schon eine Sonderstellung als DOIF-Attribut hat, da es bereits ein Attribut Userreadings für ähnliche Funktionalität gibt.

So wie heute notify und DOIF als Eventhandler für Problemstellungen genannt werden, werden zukünftig möglicherweise Userreadings und "doifReadings" als Problemlösungen in einem Atemzug genannt werden.

"doifReading" ist kürzer zu schreiben als "...Readings im DOIF-Modul" denn es funktioniert ja im Gegensatz zu Userreadings nur im DOIF-Modul und das würde man am Namen des Attributes sonst nicht erkennen.

Außerdem kann man ein DOIF-Modul bauen, welches nur aus doifReadings besteht - daher Sonderstellung des Attributs.
Programmierte FHEM-Module: DOIF-FHEM, DOIF-Perl, DOIF-uiTable, THRESHOLD, FHEM-Befehl: IF

Ellert

Bevor sich der Attributname sich zu doifReadings verfestigt, aber der Modulname in Großbuchstaben geschrieben ist, sollte dann nicht auch die Großschreibung berücksichtigt werden, etwa DOIF_Readings,  readingsDOIF, readings4DOIF, userReadingsDOIF u.ä.

Die Großschreibung des Modulnamen hat einen Wiedererkennungswert, der nicht verspielt werden sollte.

Per

Die Syntax der Attribute ist "kleinGrossGross". Die Syntax der Devices wird leider nicht so stringent gehandhabt. Daher sollte man das nicht auch noch aufweichen.

Ellert

Zitat von: Per am 11 August 2017, 16:44:16
Die Syntax der Attribute ist "kleinGrossGross". Die Syntax der Devices wird leider nicht so stringent gehandhabt. Daher sollte man das nicht auch noch aufweichen.

"kleinGrossGross" ist keine festgeschriebene Syntax, eher eine geübte Paxis, von der es genügend Abweichungen gibt, beispielsweise in den Modulen Text2Speech, DbLog, SOMFY, IT  und namentlich die Attribute IODev, event-on.*, userattr , um einige zu nennen und viele globale Attribute. Da gibt es nichts mehr aufzuweichen, es ist schon eine butterweiche Syntax. Insofern ist die Großschreibung des Modulnamen im Attributsnamen leicht zu tolerieren, zumal sie dazu dient den Wiedererkennungswert des Wortbildes DOIF aufrecht zu erhalten.

Die geübte Praxis spricht eher gegen die Kleinschreibung von DOIF im Attributnamen, z.B. wird die Schreibung des Modulnamen beibehalten in DOIFtoolsEventMonitorInDOIF und ITrepetition.

Damian

Dann schlage ich "DOIF_Readings" vor. DOIFReadings bzw. DOIFreadings liest sich schlecht.
Programmierte FHEM-Module: DOIF-FHEM, DOIF-Perl, DOIF-uiTable, THRESHOLD, FHEM-Befehl: IF

Ellert

Zitat von: Damian am 11 August 2017, 20:00:06
Dann schlage ich "DOIF_Readings" vor. DOIFReadings bzw. DOIFreadings liest sich schlecht.
Damit wäre aus meiner Sicht ein guter Kompromiss gefunden, der die vorgebrachten Ansichten und die übliche Praxis angemessen berücksichtigt.

Tsturm

Hi Damian,

bin mir nicht sicher, aber das Feature ist noch nicht im Standard, oder?

Benutze gerade userreading, aber die werden nur beim Feuern des DOIF aktualisiert  - aber nicht während der Laufzeit des DOIF (will über Hourcounter die aktuelle Laufzeit des Regners mit verwenden, der über eine Zeitspanne im DOIF an- und ausgeschaltet wird)

VG timmo

Damian

Zitat von: Tsturm am 06 September 2017, 22:20:28
Hi Damian,

bin mir nicht sicher, aber das Feature ist noch nicht im Standard, oder?

Benutze gerade userreading, aber die werden nur beim Feuern des DOIF aktualisiert  - aber nicht während der Laufzeit des DOIF (will über Hourcounter die aktuelle Laufzeit des Regners mit verwenden, der über eine Zeitspanne im DOIF an- und ausgeschaltet wird)

VG timmo

Leider noch nicht. Das Attribut wird ja jetzt DOIF_Readings heißen.
Programmierte FHEM-Module: DOIF-FHEM, DOIF-Perl, DOIF-uiTable, THRESHOLD, FHEM-Befehl: IF