syssi / esphome-pipsolar

ESPHome component to monitor and control a pipsolar inverter via RS232
Apache License 2.0
78 stars 35 forks source link

Added QPGS sensors for parallel or three phase #101

Closed gio-dot closed 6 months ago

gio-dot commented 6 months ago

@syssi I tried to decode commands in three phases (or parallel) configuration and implement it in PIP8048 component. I'm not a software expert, but i tried to use the correct syntax following existing commands. I added all this section to the component (fig.1). I also tested QPGS commands on my PIP8048 three phase configuration and they reply as in the protocol (fig.2).

Esphome firmware using modified component compile and load correctly, all original component sensors works, but unfortunately the part that i added doesn't work. Sensors are NA. It seems that QPGSn command is sent but there is no reply (fig.3)

Can someone check if i missing something adding new command? This modification can be useful for those that want to use parallel or three phase configuration... Attached there is my modified component. Please tell me a better way to load it (maybe a new branch or a pull request?)

image

2024-01-03 20_31_57-Free Device Monitoring Studio

image

syssi commented 6 months ago

Does it work now?

gio-dot commented 5 months ago

Does it work now?

I'm sorry, i closed my pr because i noticed Too late that a previous pr from another user already included qpgs sensors, so please ignore my unuseful work 😅🤯. https://github.com/syssi/esphome-pipsolar/pull/90