Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

restart Ahoy mit MqTT Befehl #1667

Closed
Tracked by #1199
DanielR92 opened this issue Jun 7, 2024 · 7 comments
Closed
Tracked by #1199

restart Ahoy mit MqTT Befehl #1667

DanielR92 opened this issue Jun 7, 2024 · 7 comments
Assignees
Labels
enhancement New feature or request fixed dev fixed

Comments

@DanielR92
Copy link
Collaborator

DanielR92 commented Jun 7, 2024

Es wurde gewünscht (wie im Titel zu lesen).
Sobald MQTT umgebaut wurde (Queue), kann dies auch mit aufgenommen werden.

@DanielR92 DanielR92 added the enhancement New feature or request label Jun 7, 2024
@lumapu
Copy link
Owner

lumapu commented Jun 9, 2024

Queue ist doch schon umgebaut, oder auf was beziehst du dich?

@DanielR92
Copy link
Collaborator Author

Na, ich kann das aktuell nicht einbauen da pubMqtt.h::onMessage die Funktion es noch nicht hergibt die Daten korrekt auszuwerten.

Ich bin hier noch etwas unzufrieden wie es aktuell gelöst ist.

@lumapu
Copy link
Owner

lumapu commented Jun 9, 2024

beziehst du dich auf Versionen vor 0.8.124 oder ab .124 und neuer?
In der .124 habe ich die Queue bereits umgebaut, wie es @fumanchi empfohlen hat. Also bei onMessage(...) wird der Inhalt in eienr std::queue eingereiht. Diese Queue wird dann in der loop durch eine leere ausgetauscht (std::swap) und anschließend die getauschte geleert. Alles ist per mutex threadsafe gestaltet.

@lumapu lumapu added the fixed dev fixed label Aug 4, 2024
lumapu added a commit that referenced this issue Aug 4, 2024
* add timestamp to JSON output #1707
* restart Ahoy using MqTT #1667
@lumapu
Copy link
Owner

lumapu commented Aug 4, 2024

Ahoy subscribed hier ein neues topic. Wichtig für ioBroker Nutzer: nach dem über das topic ein reboot getriggert wurde, muss man den Wert wieder löschen, sonst befindet sich Ahoy in einer Bootschleife - das liegt daran, dass ioBroker den Wert beim Subscriben gleich wieder rausschickt.

grafik

@knickohr
Copy link

knickohr commented Aug 8, 2024

Verbesserungsvorschlag : Ahoy setzt diesen Wert vor dem Reboot selbst zurück 😉

Es ist eine Frage der Zeit bis „ein Dummer“ kommt und genau deswegen Issues aufmacht. Nur weil er vergessen hat das sein HA oder weißgottwas default retained Topics macht.

@lumapu
Copy link
Owner

lumapu commented Aug 8, 2024

ja habe auch schon darüber nachgedacht, finde es aber eigentlich ein Fehler von ioBroker und nicht von Ahoy.

@knickohr
Copy link

knickohr commented Aug 9, 2024

Das ist doch genau das gleiche wie mit den Werten die über Nacht nicht verfügbar sind und im HA n/a ausgeben 😞

@lumapu lumapu closed this as completed Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request fixed dev fixed
Projects
None yet
Development

No branches or pull requests

3 participants