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

MQTT-Topic "set/current" wird falsch mit "phases_to_use" beschrieben #1486

Merged
merged 2 commits into from
Apr 2, 2024

Conversation

IAmKonni
Copy link
Contributor

Die besagte Zeile ist in sich falsch und zudem unnötig, da in den Zeilen 105 und 106 set_current bereits verarbeitet und auf das MQTT-Topic gepusht wird.

Die Zeile führt dazu, dass bei "trigger_phase_switch" Log-Einträge in der nachfolgenden Art im main.log erzeugt werden:

2024-03-16 08:23:57,828 - {helpermodules.setdata:338} - {ERROR:Setdata} - Payload ungültig: Topic openWB/set/chargepoint/0/set/current, Payload 3 liegt in keinem der angegebenen Wertebereiche.
2024-03-16 08:25:17,937 - {helpermodules.setdata:338} - {ERROR:Setdata} - Payload ungültig: Topic openWB/set/chargepoint/0/set/current, Payload 1 liegt in keinem der angegebenen Wertebereiche.
2024-03-16 10:24:57,003 - {helpermodules.setdata:338} - {ERROR:Setdata} - Payload ungültig: Topic openWB/set/chargepoint/0/set/current, Payload 3 liegt in keinem der angegebenen Wertebereiche.
2024-03-16 10:26:58,111 - {helpermodules.setdata:338} - {ERROR:Setdata} - Payload ungültig: Topic openWB/set/chargepoint/0/set/current, Payload 1 liegt in keinem der angegebenen Wertebereiche.
2024-03-16 14:59:57,518 - {helpermodules.setdata:338} - {ERROR:Setdata} - Payload ungültig: Topic openWB/set/chargepoint/0/set/current, Payload 3 liegt in keinem der angegebenen Wertebereiche.
2024-03-16 15:01:17,414 - {helpermodules.setdata:338} - {ERROR:Setdata} - Payload ungültig: Topic openWB/set/chargepoint/0/set/current, Payload 1 liegt in keinem der angegebenen Wertebereiche.

This line is wrong and senseless. The number of phases should not been written to the mqtt topic for currents.
Currents are already set and written in lines 105 and 106.
@IAmKonni IAmKonni changed the title MQTT-Topic "set_current" wird falsch mit "phases_to_use" beschrieben MQTT-Topic "set/current" wird falsch mit "phases_to_use" beschrieben Mar 16, 2024
@benderl benderl requested a review from LKuemmel March 18, 2024 08:43
@benderl benderl added the bug Something isn't working label Mar 18, 2024
@IAmKonni
Copy link
Contributor Author

Hab mir das nochmal angesehen. Mittlerweile glaube ich, dass die Zeile nicht falsch sowie "zu viel" war, sondern das hier eigentlich ein anderes Topic beschrieben werde sollte.

Vielleicht wäre das so in der Art korrekt gewesen:

pub_single(f"openWB/set/chargepoint/{self.hierarchy_id}/set/phases_to_use", payload=data.phases_to_use)

@LKuemmel LKuemmel added this to the 2.4.1 Step 2 milestone Apr 2, 2024
@LKuemmel LKuemmel merged commit 53ee191 into openWB:master Apr 2, 2024
1 check passed
@IAmKonni
Copy link
Contributor Author

IAmKonni commented Apr 2, 2024

@LKuemmel
Hast du meinen Kommentar noch gecheckt? Die betroffene Zeile ist zwar falsch, aber ich bin mir nicht sicher, ob da eine andere Zeile stehen sollte.

@LKuemmel
Copy link
Contributor

LKuemmel commented Apr 3, 2024

Ja, habe ich. Das passt so.

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

Successfully merging this pull request may close these issues.

3 participants