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

[Bug] MQTT - Total der ahoyDTU wird nicht im Discovery Config (homeassistant) mitgesendet #1715

Open
1 task
evilmumi opened this issue Aug 9, 2024 · 11 comments
Assignees
Labels
bug Something isn't working fixed dev fixed

Comments

@evilmumi
Copy link

evilmumi commented Aug 9, 2024

Platform

ESP32

Assembly

I did the assebly by myself

nRF24L01+ Module

nRF24L01+ plus

Antenna

circuit board

Power Stabilization

Elko (~100uF)

Connection picture

  • I will attach/upload an Image of my wiring

Version

0.8.130

Github Hash

24ceb3e

Build & Flash Method

AhoyDTU Webinstaller

Setup

grafik

Debug Serial Log output

No response

Error description

Nach einem Restart der dtu / homeassistant habe / hatte ich immer mal Probleme mit unavailable von den Sensoren in HA.
Es half immer ein manuelles drücken des SEND Discovery Config (homeassistant).
Dies funktioniert nun auch für alle beiden Inverter.
Diese tauchen unter homeassistant/sensor/ auf.
Leider nicht (mehr) das ahoydtu ansich - ich habe kürzlich von 0.8.85 auf .130 geupdated. Ich weiß nicht, ob es damit zusammen hängen könnte. Ich habe erstmal den issue eröffnet bevor ich wieder downgrade.

@evilmumi evilmumi added the bug Something isn't working label Aug 9, 2024
@evilmumi
Copy link
Author

evilmumi commented Aug 9, 2024

Update: ein Downgrade auf 0.8.85 hilft - das discovery sendet homeassistant/sensor/ahoydtu32_TOTAL.

@evilmumi
Copy link
Author

hat wer einen Tipp hierzu? das downgraden nur für den Publish ist nervig. Ich bin aber nicht so der mqtt experte - aber kann ich nicht die Sachen selber publishen? jemand eine Idee?

@martin-st-81
Copy link

martin-st-81 commented Sep 21, 2024

Ist per MQTT in HomeAssistant verfügbar, aufgrund des Bugs muss es manuell in der mqtt.yaml hinzugefügt werden.
Ich habe es wie u.a. eingebunden:

sensor:
- name: Aktuelle Leistung Solar
state_topic: "inverter/total/P_AC"
qos: 2
unit_of_measurement: "W"
unique_id: "AktuelleLeistungSolar"
state_class: measurement
device_class: energy
icon: mdi:solar-power

2024-09-21 17_00_20-Studio Code Server – Home Assistant

@lumapu lumapu added the fixed dev fixed label Sep 21, 2024
lumapu added a commit that referenced this issue Sep 21, 2024
* fix NTP related issues #1748 #1752
* fix MqTT discovery total #1715
* upgrade webserver version for ESP32 devices
@evilmumi
Copy link
Author

welchen Bug meinst du?

und wie bekomme ich das hin, dass es regelmäßig gesendet wird? Manchmal ist es nach HA restart /reboot/mqtt server reboot weg!? ich spiele mal 145 ein

@lumapu
Copy link
Owner

lumapu commented Sep 23, 2024

Ich bin kein homeassistant-Nutzer, daher kann ich die Frage nicht beantworten. Muss man das wirklich regelmäßig senden?

@martin-st-81
Copy link

die Übermittlung mittels MQTT nach HomeAssistant erfolgte korrekt (ersichtlich in MQTT Explorer im HomeAssistant). Nur wurde es nicht autom. eingebunden. Dies wurde mit Version 145 jetzt behoben.

@evilmumi
Copy link
Author

wie gesagt, ich beobachte nun mit 145 und melde mich, wenn es nicht mehr geht, was ich getan habe. danke soweit

@stefan123t
Copy link
Collaborator

@evilmumi hast Du das Problem mit der aktuellen v0.8.152 noch oder können wir das issue als gelöst schließen ?

@evilmumi
Copy link
Author

evilmumi commented Nov 4, 2024

Ich habe gerade die .145 drauf
Wollte gerade auf 152 updaten. Da sagt er failed. Weiß nicht warum. Nach einem reboot macht er aber auch hier nicht nochmal eine home Assistent Ankündigung somit sind die Werte alle leer.
Was dann aber in 145 klappt ist nen händisch send config senden. Ist aber bisschen schade dass es nicht automatisch kommt.

@stefan123t
Copy link
Collaborator

@evilmumi die Home Assisstant auto-config schickt die AhoyDTU m.W. nur auf Anforderung durch den Button.
Automatisch wird die m.W. gar nicht versendet.

Wie oft bzw. zu welchen Anlässen sollte AhoyDTU die sendDiscoveryConfig() Routine denn Deiner Meinung nach ausführen ?

Und was ist mit anderen Nutzern, die keinen Home Assisstant angebunden haben, die wollen die Auto-Discovery MQTT Nachrichten ja evtl. nicht haben ?

ahoy/src/app.h

Lines 230 to 234 in 2d5d62e

void setMqttDiscoveryFlag() override {
#if defined(ENABLE_MQTT)
once(std::bind(&PubMqttType::sendDiscoveryConfig, &mMqtt), 1, "disCf");
#endif
}

@evilmumi
Copy link
Author

evilmumi commented Nov 4, 2024

Ich denke, es wäre super, wenn bei einem neustart von ahoy und oder HA dies neu gesendet wird. Bin aber nicht so tief in dem Thema um das bewerten zu können. Naja die, die kein MQTT + HA haben könnten es ja per checkbox abwählen.
Ich überlege auch gerade, ob es eine Möglichkeit gibt, dass ich nach einem HA Neustart die routine bei ahoy anstarten könnte - das wäre ja auch nen (Super.)Weg (für mich)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed dev fixed
Projects
None yet
Development

No branches or pull requests

4 participants