kbialek / deye-inverter-mqtt

Reads Deye solar inverter metrics and posts them over MQTT
Apache License 2.0
230 stars 50 forks source link

Non-numeric serial number #213

Open MagiX13 opened 4 days ago

MagiX13 commented 4 days ago

For some reason, my Deye SUN-M80G3-EU-Q0 (obtained in Nov 2023, never upgrade the firmware thus running MW3_16U_5406_2.27) has a non-numeric serial number, ending with BD.

I made some local changes to accommodate this, but was wondering if there are more people with this/there is a need for a pull request - I think it amounts to 3-4 lines that I had to change. I had it running with these changes since March already and it seems stable. Today I decided to update the docker repo and saw that this is still not supported and would be happy to submit a PR.

kbialek commented 3 days ago

hi! I think it was a bad design decision to model the serial number as integer, so if you would like to submit a PR fixing this, that would be great.