GivEnergy / giv_tcp

TCP connection (from inverter) and MQTT implementation
30 stars 41 forks source link

Only 1 battery discovered #22

Open holdestmade opened 1 year ago

holdestmade commented 1 year ago

I have a AC3.0 inverter and 2x 5.2 batteries. This shows correctly in the GE dashboard.

giv_tcp only discovers 1 of the batteries

G6EJD commented 1 year ago

In the Configuration did you select 2 batteries?

holdestmade commented 1 year ago

Hi, yes sure did

holdestmade commented 1 year ago

This is in my logs, seems one of the serial number is just spaces (asterisks below are spaces in the logs)

2022-12-21 18:50:26,623 - read - [INFO] - Building battery output: 2022-12-21 18:50:26,623 - read - [INFO] - Battery BJ123456789 added 2022-12-21 18:50:26,624 - read - [INFO] - Building battery output: 2022-12-21 18:50:26,624 - read - [INFO] - Battery ***** added 2022-12-21 18:50:26,629 - read - [INFO] - Running the data cleansing process

holdestmade commented 1 year ago

If I select 3 batteries in the config the missing battery appears but no MQTT device is created. As you can see there is still a battery with no serial number. Only the BJ111111112 device is created

2022-12-22 08:11:10,269 - read - [INFO] - Getting Battery Details 2022-12-22 08:11:10,269 - read - [INFO] - Building battery output: 2022-12-22 08:11:10,270 - read - [INFO] - Battery BJ111111112 added 2022-12-22 08:11:10,270 - read - [INFO] - Building battery output: 2022-12-22 08:11:10,270 - read - [INFO] - Battery ***** added 2022-12-22 08:11:10,270 - read - [INFO] - Building battery output: 2022-12-22 08:11:10,270 - read - [INFO] - Battery BJ111111111 added 2022-12-22 08:11:10,270 - read - [INFO] - Building battery output: 2022-12-22 08:11:10,271 - read - [INFO] - Battery added

holdestmade commented 1 year ago

Just checked with the GE API and both batteries appear correctly:

        "connections": {
          "batteries": [
            {
              "serial": "BJ1111G112",
              "firmware_version": "3009",
              "capacity": {
                "full": 99.13,
                "design": 102
              },
              "cell_count": 16,
              "has_usb": true
            },
            {
              "serial": "BJ1111G111",
              "firmware_version": "3009",
              "capacity": {
                "full": 96.59,
                "design": 102
              },
              "cell_count": 16,
              "has_usb": true
            }
          ]