add eps registers and home assitant json #240
Conversation
jgulick48
changed the title
| FROM debian:bullseye-slim | ||
| RUN apt-get update && apt-get install -y libssl1.1 && rm -rf /var/lib/apt/lists/* | ||
| FROM debian:bookworm-slim | ||
| RUN apt-get update && apt-get install -y openssl && rm -rf /var/lib/apt/lists/* |
There was a problem hiding this comment.
Oh nice, might this help with the ssl build failures I was struggling with yesterday? The commits to master were trying to fix it, I ended up staying on an older rust compiler/clap version to work around it. I'll retry newer versions after merging this, hopefully it helps.
There was a problem hiding this comment.
Yeah, I was getting ssl errors on startup and this is what fixed it for me.
|
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. |
|
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 :) |
|
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 #249 |
3 participants
Building on #239, adding registers for the EPS data and also updating the json for the Home Assistant integration.