soc_VWID/VWEUDA: improvements#3548
Conversation
…rove field parsing logging
| topic = f"openWB/vehicle/{vehicle}/soc_module/config" | ||
| conf = os.popen(f"mosquitto_sub -C 1 -t {topic}").read() | ||
| _LOGGER.debug(f"thread loop: conf={conf}") | ||
| _type = json.loads(conf)['type'] | ||
| _LOGGER.info(f"thread loop: _type={_type}") | ||
| if _type != 'vwid': | ||
| _LOGGER.info(f"vehicle {vehicle} is not using module vwid: terminate now") | ||
| return |
There was a problem hiding this comment.
Dieser Code ist eher ein Workaround und zeigt, dass while True keine saubere Lösung ist. Gibt es eine andere Lösung für das Thread-Handling, konsistent zum restlichen Code?
There was a problem hiding this comment.
HalloLena, Ich mache das noch schöner, Kann ich die Config des EV bzw. dessen SoC-Modul Config intern abfragen?
Das "while True" werde ich durch eine bessere Bedingung erstzen aber momentan fehlt mir die Zeit.
Es wäre schön wenn es erst mal gemerged würde damit die Tester weitermachen können.
There was a problem hiding this comment.
Das SoC-Modul soll möglichst unabhängig von der Fahrzeug-Klasse Ev sein.
There was a problem hiding this comment.
Hallo @rleidner,
Deine Implementierung scheint ja stabil zu laufen, sofern es valide Daten vom Portal gibt.
Für ein Patch-Release muss der Code wie oben besprochen nochmal überarbeitet werden. Hast Du dir da nochmal Gedanken zu gemacht?
There was a problem hiding this comment.
Hallo @LKuemmel , danke für die Erinnerung.
Ich habe das schon soweit fertig, ich werde den PR kurzfristig machen.
2 participants
@LKuemmel , please merge ASAP, thanks!