lxp-bridge icon indicating copy to clipboard operation
lxp-bridge copied to clipboard

Rewrite packet parsing code

Open celsworth opened this issue 11 months ago • 1 comments

This will be a long-lived PR to try and address the list in https://github.com/celsworth/lxp-bridge/issues/245

I'll create smaller separate PRs and merge into this to try and keep it manageable.

This adds a new configuration option, publish_inputs_all_trigger. This lives at the root of the config (ie next to loglevel). By default, this is 80. If you have an inverter which publishes 4 sets of inputs data packets (like newer EG4 inverters with generator inputs) you probably need to bump this to 120 to get inputs/all sent with all the correct data.

However, with this release I'm heavily deprecating the MQTT messages inputs/1 (and 2 / 3 / 4) and inputs/all. Please do not use them in any new projects. Instead, use the individual messages published when each register is received. These have topics like input/status/parsed, input/e_pv_day_1/parsed etc and the payloads are equivalent to the old ones.

TODO:

  • re-add database support, it is missing
  • params messages are not implemented yet, but I don't think I'll bother as they weren't that useful.

celsworth avatar Mar 13 '24 16:03 celsworth

This looks great, can't wait for it to land!

presto8 avatar Mar 15 '24 13:03 presto8