98_JsonMod.pm - typo in commandref

Begonnen von betateilchen, 01 März 2022, 17:51:39

Vorheriges Thema - Nächstes Thema

yersinia

Kann man denn nicht Schedule::Cron (ja, ich kann eine Suchmaschine benutzen ^^) nutzen? Oder weicht deine Implementierung ab?
viele Grüße, yersinia
----
FHEM 6.4 (SVN) on RPi 4B with RasPi OS Trixie (perl 5.40.1) | FTUI
nanoCUL->2x868(1x ser2net)@tsculfw, 1x433@Sduino | MQTT2 | Tasmota | ESPEasy
VCCU->14xSEC-SCo, 7xCC-RT-DN, 5xLC-Bl1PBU-FM, 3xTC-IT-WM-W-EU, 1xPB-2-WM55, 1xLC-Sw1PBU-FM, 1xES-PMSw1-Pl

herrmannj


betateilchen

#17
Zitat von: herrmannj am 04 März 2022, 16:14:26
@betateilchen, ja weil Kommata (so wie Wochentage) nicht berücksichtigt werden.

Nur mal so (ja ich habe Deinen Folgekommentar gelesen): meine Intervalldefinition funktioniert grundsätzlich von 20-07:59 wie gewünscht.
Nur der erste Zeitpunkt stimmt nicht. Soll: 20:01, Ist: 20:59
Danach wird um 21:01 Uhr korrekt weitergearbeitet.

"1,30,59 0-7,20-23 * * *" funktioniert auch, dann wird der erste Ausführungszeitpunkt auf 20:30 Uhr berechnet.
Es scheint so, dass der erste Eintrag in der Komma-Liste verlorengeht.
----------
Es ist mir egal, wer Dein Vater ist.
Solange ich hier angele, läufst Du hier nicht übers Wasser!
----------

betateilchen

Zitat von: yersinia am 04 März 2022, 16:31:36
aber mal [OT]: die cron-Notation und die dazugehörige Funktion hätte ich -neben JsonMod- auch gern für at, DOIF, HTTPMOD, Calendar, ABFALL und alles andere was irgendwie mit Interval arbeitet.

Das hätte ich auch gerne.

Zitat von: herrmannj am 04 März 2022, 16:43:27
CRON (.pm oder was auch immer) so zur Verfügung zu stellen wäre mein Ziel. Ob die maintainer der genannten Module das dann nutzen, das liegt nicht bei mir.

Naja, es muss halt möglichst einfach zu verwenden sein. Es gab vor einiger Zeit schonmal den Versuch, mit computeAlignTime() in fhem.pl die Berechnung von align-Attributen zu vereinfachen/vereinheitlichen. Aber ausser dem at selbst kenne ich kein Modul, das diese Funktion tatsächlich benutzt.
----------
Es ist mir egal, wer Dein Vater ist.
Solange ich hier angele, läufst Du hier nicht übers Wasser!
----------

herrmannj

#19
Zitat von: betateilchen am 04 März 2022, 17:20:51
Nur mal so (ja ich habe Deinen Folgekommentar gelesen): meine Intervalldefinition funktioniert grundsätzlich von 20-07:59 wie gewünscht.
Nur der erste Zeitpunkt stimmt nicht. Soll: 20:01, Ist: 20:59
Danach wird um 21:01 Uhr korrekt weitergearbeitet.

"1,30,59 0-7,20-23 * * *" funktioniert auch, dann wird der erste Ausführungszeitpunkt auf 20:30 Uhr berechnet.
Es scheint so, dass der erste Eintrag in der Komma-Liste verlorengeht.
du hast recht, Kommata hatte ich bereits implementiert. ist dann offensichtlich ein bug, danke fürs aufspüren, ich arbeite dran

betateilchen

Ja, mach Dir keinen Streß. Ich bin da auch nur zufällig drauf gestoßen, ist aber eher unkritisch.

Eigentlich wollte ich nur einen Tippfehler melden  :D

----------
Es ist mir egal, wer Dein Vater ist.
Solange ich hier angele, läufst Du hier nicht übers Wasser!
----------

herrmannj

den ignoriere ich erstmal  :D hat ja zu zwei (echten) bug meldungen geführt, da gilts nachzuarbeiten

betateilchen

Bezüglich des ignorierten Tippfehler könnte man auch darüber nachdenken, ob "interval" die richtige Bezeichnung für das Attribut ist. Mit crontab-ähnlichen Einträgen werden ja nicht Ausführungsintervalle definiert, sondern Ausführungszeitpunkte. Vielleicht wäre "schedule" oder "crontab" als Attributname klarer in ihrer Bedeutung.
----------
Es ist mir egal, wer Dein Vater ist.
Solange ich hier angele, läufst Du hier nicht übers Wasser!
----------

betateilchen

Ich habe keine Ahnung wieso, aber die eingebauten Änderungen funktionieren heute nicht mehr.

*/10 7-20 * * *

liefert aktuell 07:20 als nächsten Ausführungszeitpunkt.
Aber vielleicht übersehe ich auch gerade irgendwas aufgrund der späten Uhrzeit.
----------
Es ist mir egal, wer Dein Vater ist.
Solange ich hier angele, läufst Du hier nicht übers Wasser!
----------

herrmannj

Danke fürs bescheid sagen. Steck nicht zu viel rein, sobald die kommende version fertig ist, wären ausführliche tests nochmals sehr hilfreich. da dann auch im 'at'