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

0.5.106 MQTT-Autodiscover: second sensor with new name created #805

Closed
rmayergfx opened this issue Mar 25, 2023 · 10 comments
Closed

0.5.106 MQTT-Autodiscover: second sensor with new name created #805

rmayergfx opened this issue Mar 25, 2023 · 10 comments
Assignees
Labels
fixed dev fixed question Further information is requested

Comments

@rmayergfx
Copy link

Nach dem Update auf FW v0.5.106 hat das System nun ein zusätzliches Gerät "AHOY_DTU_DF7C" angelegt das auf der gleichen IP liegt...

Downgrade auf v0.5.105 und das zusätzliche Gerät verschwindet wieder.

@Argafal
Copy link
Contributor

Argafal commented Mar 25, 2023

Ich glaube das ist ein neues Feature in 0.5.106, wenn ich das richtig verstanden habe. Statt AHOY_DTU gibt es jetzt einen eindeutigen Namen. Für die die nur ein Device haben ist es egal, aber für die die mehr als einen Testaufbau haben hilft es die verschiedenen Geräte zu unterscheiden. Im Discord gibt's glaub ich ne Anleitung wie man das umbenennen kann, falls man es wieder nur als "AHOY-DTU" haben will.

@dtuuser
Copy link

dtuuser commented Mar 25, 2023

Bei mir ist der Name geblieben und es ist in der Fritzbox kein weiterer erschienen

@Argafal
Copy link
Contributor

Argafal commented Mar 25, 2023

Ich nahm an, dass @rmayergfx vom MQTT auto discovery spricht. Vielleicht hab ich es falsch verstanden. Warten wir mal bis er/sie erklärt von welchem "System" geredet wird. Ja, der hostname für den Router ist glaube ich gleich geblieben.

Wäre das folgende ein guter Änderungsvorschlag? Man verwendet immer den eingestellten Hostname als Device name in MQTT. Der hostname kriegt als Default ein geräteabhängiges Suffix. Ein Benutzer der lieber einen anderen Hostname und Device name hat, kann das Gerät dann einfach selber in den ahoy Einstellungen umbenennen.

@rmayergfx
Copy link
Author

rmayergfx commented Mar 25, 2023

Also in den Ahoy Settings steht bei mir für MQTT nicht der default "inverter" sondern M800. Nun ist ein 2tes Device im MQTT aufgetaucht mit dem Namen "AHOY_DTU_DF7C" und dieses weitere Gerät hat 4 Sensoren. Ich habe aber nur eine AhoyDTU im Netzwerk und nun in HA ein weiteres MQTT Gerät das eigentlich gar nicht existieren dürfte, da es ja bereits das Gerät gibt.
Der Inverter hat im Setup den Namen TSUN_M800 und dort sind auch soweit alle Sensoren vorhanden.

Somit habe ich
TSUN_M800
AHOY_DTU_DF7C

die beide in HA unter Gerätieinformationen > Besuchen auf das gleiche AhoyDTU mit der passenden IP-Adresse gehen, was verstänldich ist, da ich ja nur 1 Device besitze.

Was mir hier jedoch fehlt sind nach wie vor die MQTT Informationen zu Firmware und HWPartId, diese sind nur per MQTT Explorer sichtbar, tauchen jedoch nirgendwo in HA auf.

@lumapu
Copy link
Owner

lumapu commented Mar 26, 2023

Hier geht es um Autodiscover. Das zweite Gerät sind die Totalwerte, also Summen über alle Inverter. Evtl. sollten wir das publishen der Totalwerte nur bei Multi-Inverter-Setups machen.
Stört das Gerät wirklich?

@lumapu lumapu self-assigned this Mar 26, 2023
@lumapu lumapu added the question Further information is requested label Mar 26, 2023
@lumapu lumapu changed the title FW v0.5.106 bug second device with new name created 0.5.106 MQTT-Autodiscover: second sensor with new name created Mar 26, 2023
@rmayergfx
Copy link
Author

rmayergfx commented Mar 26, 2023

Ja, da es nur zur Verwirrung führt und nicht wirklich eindeutigt in der Beschreibung dem jeweiligen Inverter zuzuordnen ist. Ergibt aus meiner Sicht keinen Sinn ein neues zusätzliches Gerät beim Single Setup zu sehen. Auch im Multi-Inverter-Setup finde ich es einfacher wenn jemand Totalwerte benötigt sich dazu einen Helper zu erstellen. Summierung kann doch inzwischen glaube ich jede eingesetzte Software.

@beegee3
Copy link
Contributor

beegee3 commented Mar 26, 2023

Totalwerte liefern nur bei Multi-Inverter-Setups neues, machen so gesehen nur dort Sinn.
Das 'Total' Gerät vielleicht mit dem Device Name aus dem Setup + '_Total' benennen, dann ist die Inverter Zuordnung klar.

@rmayergfx
Copy link
Author

Wofür ein "Total" Gerät bei einem Single Setup? Alle Werte dort wie gehabt einpflegen und für ein Multi-Inverter-Setup eine Option in der Firmware einpflegen das dort ein zustätzliches Gerät "Multi-Total" optional aktiviert werden kann. Dann ist es eindeutig und führt bei Single Setups nicht zu Verwirrungen.

@beegee3
Copy link
Contributor

beegee3 commented Mar 26, 2023

Wofür ein "Total" Gerät bei einem Single Setup? Alle Werte dort wie gehabt einpflegen und für ein Multi-Inverter-Setup eine Option in der Firmware einpflegen das dort ein zustätzliches Gerät "Multi-Total" optional aktiviert werden kann. Dann ist es eindeutig und führt bei Single Setups nicht zu Verwirrungen.

'Total' nur bei Multi-Inverter, so meinte ich das! Dann aber sollte der 'Total' Gerätename nicht fix sein, sondern von der Ahoy-DTU abhängen, sonst gibt es Probleme, wenn jemand mehr als eine DTU einsetzt. Und da bietet sich der im Setup konfigurierbare Device Name an (mit angefügtem "_Total"). Ist leichter zu identifizieren, als das jetzige "AHOY_DTU_XXYY" mit XXYY aus der MAC erzeugt.

@lumapu
Copy link
Owner

lumapu commented Mar 26, 2023

@beegee3 @rmayergfx : Umbau ist schon im Gange, bin gerade beim Test

lumapu added a commit that referenced this issue Mar 26, 2023
* fix: no MQTT `total` sensor for autodiscover if only one inverter was found #805
* fix: MQTT `total` renamed to `device_name` + `_TOTOL` for better visibility #805
@lumapu lumapu added the fixed dev fixed label Mar 26, 2023
@lumapu lumapu closed this as completed Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed dev fixed question Further information is requested
Projects
None yet
Development

No branches or pull requests

5 participants