search

celsworth / lxp-bridge

A bridge to MQTT/InfluxDB/Postgres for communications with LuxPower inverters
MIT License
9 stars 14 forks source link

add eps registers and home assitant json #240

Closed jgulick48 closed 5 months ago

jgulick48 commented 6 months ago

Building on #239, adding registers for the EPS data and also updating the json for the Home Assistant integration.

celsworth commented 6 months ago

This looks good, the only thing thats starting to fall behind is probably the database storage, but I'm not sure if anyone really cares.

I thought about removing it entirely because the sqlx library is a pain in the ass anyway..

Edit: oh actually might be good to get these in Influx though (which is a lot easier). I can do that separately.

presto8 commented 6 months ago

Thanks for doing this! I have a new inverter setup and am figuring out how to balance the split phases. I saw on the web dashboard "EPS L1N" and "L2N" and was wondering how to get those in lxp-bridge. Then I saw this pull request :)

presto8 commented 5 months ago

Unfortunately this PR seems to have broken lxp-bridge on my system. I'm not sure how to go about debugging it, but what happens is if I run this new build, the active EPS power stops being updated. If I revert back to the previous commit, it works. I'll try to collect more debug info, just an initial observation.

celsworth commented 5 months ago

Unfortunately this PR seems to have broken lxp-bridge on my system. I'm not sure how to go about debugging it, but what happens is if I run this new build, the active EPS power stops being updated. If I revert back to the previous commit, it works. I'll try to collect more debug info, just an initial observation.

I'm addressing this in https://github.com/celsworth/lxp-bridge/pull/249