US3000 parsing failure

[Vards0]

Hi.
I'm getting the following eror when I import the library:-

>>> import pylontech Traceback (most recent call last): File "<stdin>", line 1, in <module> File "pylontech.py", line 6 def _decode(self, obj, context, path) -> bytes:

Any ideas?

Thanks

[Frankkkkk]

Which version of python are you running ?

…

On October 17, 2021 8:14:37 AM GMT+02:00, Vards0 ***@***.***> wrote: Hi. I'm getting the following eror when I import the library:- `>>> import pylontech Traceback (most recent call last): File "<stdin>", line 1, in <module> File "pylontech.py", line 6 def _decode(self, obj, context, path) -> bytes: ` Any ideas? Thanks -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: #4

[Frankkkkk]

You need to install the two libs (with pip, pipenv, or whatever) specified here: https://github.com/Frankkkkk/python-pylontech#dependencies Cheers

…

On October 17, 2021 10:04:31 AM GMT+02:00, Vards0 ***@***.***> wrote: I was running version 2.7. I have upgraded to version 3.5. Now I am getting this error:- `Python 3.5.3 (default, Apr 5 2021, 09:00:41) [GCC 6.3.0 20170516] on linux Type "help", "copyright", "credits" or "license" for more information. >>> import pylontech Traceback (most recent call last): File "<stdin>", line 1, in <module> File "/pylontech/pylontech.py", line 2, in <module> import construct ImportError: No module named 'construct' ` -- You are receiving this because you commented. Reply to this email directly or view it on GitHub: #4 (comment)

[Vards0]

Thanks very much

I was running ver 2.7.

I'm now running ver 3.5. I've installed both dependencies but now I am getting this error:-

`Python 3.5.3 (default, Apr 5 2021, 09:00:41)
[GCC 6.3.0 20170516] on linux
Type "help", "copyright", "credits" or "license" for more information.

import pylontech
p = pylontech.Pylontech()
print(p.get_values())
Traceback (most recent call last):
File "", line 1, in
File "/pylontech/pylontech.py", line 211, in get_values
f = self.read_frame()
File "/pylontech/pylontech.py", line 170, in read_frame
f = self._decode_hw_frame(raw_frame=raw_frame)
File "/pylontech/pylontech.py", line 151, in _decode_hw_frame
assert got_frame_checksum == int(frame_chksum, 16)
ValueError: invalid literal for int() with base 16: b''`

[Frankkkkk]

I should add more debug verbosity, but in the meantime you could follow #2 (comment) which was the same problem

What is your battery topology ?

[Vards0]

My battery topology is 4x US3000

[Frankkkkk]

Okay, so maybe you've got the same problem as @stuartornum .

Can you give me a raw frame example by editing _decode_hw_frame:

def _decode_hw_frame(self, raw_frame: bytes) -> bytes:
  print(raw_frame)

and launching get_values again ?

That way we can try and support the US3000 batteries alone.

[stuartornum]

I managed to get my setup working by having the US2000 as the primary and the US3000 secondary. Although, I still have some weirdness when the SOC of the US3000 is above ~89% it reports as some very lower number (when I know for a fact it’s not!) and when it dips below 89% it reports fine again….? Very stalrange. Here’s my grafana graph https://ibb.co/yV2N2B9

[Vards0]

Hi Frankkkkk...

Sorry for the delay in getting back to you.

I edited the code to be as follows:-

    def _decode_hw_frame(self, raw_frame: bytes) -> bytes:
        print(raw_frame)
        # XXX construct
        frame_data = raw_frame[1:len(raw_frame) - 5]
        frame_chksum = raw_frame[len(raw_frame) - 5:-1]

        got_frame_checksum = Pylontech.get_frame_checksum(frame_data)
        assert got_frame_checksum == int(frame_chksum, 16)

        return frame_data

I'm not 100% sure if this is what I needed to do. Please correct me if I am wrong.

Now when I run the get_values, I am getting the below:-

[GCC 6.3.0 20170516] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import pylontech
>>> p = pylontech.Pylontech()
>>> print(p.get_values())
b'~2002460061DC11040F0CFD0CFC0CFC0CFB0CFC0CFB0CFD0CFC0CFC0CFB0CFA0CFD0CFB0CFE0CFA050BE10BCD0BCD0BCD0BCD0000C2C1FFFF04FFFF002F00EFEC0121100F0CEB0CEB0CEB0CEA0CEA0CEC0CEB0CEB0CE90CE80CE60CE90CE90CEA0CE8050BE10BCD0BCD0BCD0BCDFFBCC1B2FFFF04FFFF002800F2D00121100F0CE80CE90CEA0CEA0CEA0CE90CEA0CEA0CEB0CEC0CEB0CEB0CEB0CEA0CEA050BE10BC30BC30BC30BC3FFB7C1B8FFFF04FFFF007100E7400121100F0CE90CEC0CEB0CEA0CEA0CEB0CE90CE80CEA0CEA0CEA0CEB0CEC0CEA0CEA050BD70BC30BC30BC30BB9FFBBC1B9FFFF04FFFF006B00ED080121108D63\r'

@stuartornum, are you using mqtt to bring your values into grafana?

[stuartornum]

@stuartornum, are you using mqtt to bring your values into grafana?

Hi @Vards0 ! I'm using @Frankkkkk code (thank you again!) on a RPi3 polling every minute via cron, this then dumps the output into an nginx html directory... which Prometheus then comes along and pulls the data out for Grafana. I'm using Prometheus for other stuff like Shelly.cloud devices.

[Vards0]

Hi Frank

I realised that I needed to change to '/dev/ttyUSB1'

I think I'm now getting more meainingful output:-

Type "help", "copyright", "credits" or "license" for more information.
>>> import pylontech
>>> p = pylontech.Pylontech()
>>> print(p.get_values())
b'~2002460061DC11040F0CFD0CFC0CFC0CFB0CFC0CFB0CFD0CFC0CFC0CFB0CFA0CFD0CFB0CFE0CFA050BE10BCD0BCD0BCD0BCD0000C2C1FFFF04FFFF002F00EFEC0121100F0CEB0CEB0CEB0CEA0CEA0CEC0CEB0CEB0CE90CE80CE60CE90CE90CEA0CE8050BE10BCD0BCD0BCD0BCDFFBCC1B2FFFF04FFFF002800F2D00121100F0CE80CE90CEA0CEA0CEA0CE90CEA0CEA0CEB0CEC0CEB0CEB0CEB0CEA0CEA050BE10BC30BC30BC30BC3FFB7C1B8FFFF04FFFF007100E7400121100F0CE90CEC0CEB0CEA0CEA0CEB0CE90CE80CEA0CEA0CEA0CEB0CEC0CEA0CEA050BD70BC30BC30BC30BB9FFBBC1B9FFFF04FFFF006B00ED080121108D63\r'

[Frankkkkk]

Hi, thanks for the debug output ! I'll take a look tonight hopefully and try to adapt the data structure :-)

Cheers

[Vards0]

Thanks Frank

I also noticed this in the Pylontech RS485 manual.

I'm not sure if this will be of any use to you or not. It goes right over my head.

Cheers

[Frankkkkk]

You were right @Vards0 Indeed I completely missed this part. Here's the output with the correct parsing with the PDU you gave me above:

    NumberOfModules = 4
    Module = ListContainer: 
        Container: 
            NumberOfCells = 15
            CellVoltages = ListContainer: 
                3.325
                3.324
                3.324
                3.323
                3.324
                3.323
                3.325
                3.324
                3.324
                3.323
                3.322
                3.325
                3.323
                3.326
                3.322
            NumberOfTemperatures = 5
            AverageBMSTemperature = 30.41
            GroupedCellsTemperatures = ListContainer: 
                30.21
                30.21
                30.21
                30.21
            Current = 0.0
            Voltage = 49.857
            Power = 0.0
            RemainingCapacity = 65.535
            UserDefinedItems = 4
            TotalCapacity = 65.535
            CycleNumber = 47
            US3000 = Container: 
                Remaining = 61.42
                Total = 74.0
        Container: 
            NumberOfCells = 15
            CellVoltages = ListContainer: 
                3.307
                3.307
                3.307
                3.306
                3.306
                3.308
                3.307
                3.307
                3.305
                3.304
                3.302
                3.305
                3.305
                3.306
                3.304
            NumberOfTemperatures = 5
            AverageBMSTemperature = 30.41
            GroupedCellsTemperatures = ListContainer: 
                30.21
                30.21
                30.21
                30.21
            Current = -6.8
            Voltage = 49.586
            Power = -337.1848
            RemainingCapacity = 65.535
            UserDefinedItems = 4
            TotalCapacity = 65.535
            CycleNumber = 40
            US3000 = Container: 
                Remaining = 62.16
                Total = 74.0
        Container: 
            NumberOfCells = 15
            CellVoltages = ListContainer: 
                3.304
                3.305
                3.306
                3.306
                3.306
                3.305
                3.306
                3.306
                3.307
                3.308
                3.307
                3.307
                3.307
                3.306
                3.306
            NumberOfTemperatures = 5
            AverageBMSTemperature = 30.41
            GroupedCellsTemperatures = ListContainer: 
                30.11
                30.11
                30.11
                30.11
            Current = -7.3
            Voltage = 49.592
            Power = -362.0216
            RemainingCapacity = 65.535
            UserDefinedItems = 4
            TotalCapacity = 65.535
            CycleNumber = 113
            US3000 = Container: 
                Remaining = 59.2
                Total = 74.0
        Container: 
            NumberOfCells = 15
            CellVoltages = ListContainer: 
                3.305
                3.308
                3.307
                3.306
                3.306
                3.307
                3.305
                3.304
                3.306
                3.306
                3.306
                3.307
                3.308
                3.306
                3.306
            NumberOfTemperatures = 5
            AverageBMSTemperature = 30.31
            GroupedCellsTemperatures = ListContainer: 
                30.11
                30.11
                30.11
                30.01
            Current = -6.9
            Voltage = 49.593
            Power = -342.1917
            RemainingCapacity = 65.535
            UserDefinedItems = 4
            TotalCapacity = 65.535
            CycleNumber = 107
            US3000 = Container: 
                Remaining = 60.68
                Total = 74.0
    TotalPower = -1041.3981
    StateOfCharge = 1.0

Could you try the code on the fvd/us3000 branch (MR here #5 ) and tell me if it works for you ? I'm not merging just yet because I'd like to make it nice in order to have the same structure for US2000/US3000 batteries.

PS: SOC Is wrong because using the US2000 values

Cheers

[Vards0]

Frank you're a genius!!

I ran the updated code at it looks to be working a treat:-

Is there any way of separating the battery identities and their respective values so that I can pipe these into mqtt?

[GCC 6.3.0 20170516] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import pylontech
>>> print(p.get_values())
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
NameError: name 'p' is not defined
>>> p = pylontech.Pylontech()
>>> print(p.get_values())
Container:
    NumberOfModules = 4
    Module = ListContainer:
        Container:
            NumberOfCells = 15
            CellVoltages = ListContainer:
                3.531
                3.53
                3.531
                3.528
                3.531
                3.531
                3.532
                3.515
                3.533
                3.533
                3.53
                3.533
                3.533
                3.533
                3.533
            NumberOfTemperatures = 5
            AverageBMSTemperature = 30.11
            GroupedCellsTemperatures = ListContainer:
                29.91
                29.91
                30.01
                29.91
            Current = 0.0
            Voltage = 52.957
            Power = 0.0
            RemainingCapacity = 65.535
            UserDefinedItems = 4
            TotalCapacity = 65.535
            CycleNumber = 48
            US3000 = Container:
                Remaining = 74.0
                Total = 74.0
        Container:
            NumberOfCells = 15
            CellVoltages = ListContainer:
                3.523
                3.528
                3.534
                3.53
                3.531
                3.531
                3.532
                3.517
                3.535
                3.535
                3.533
                3.534
                3.534
                3.536
                3.532
            NumberOfTemperatures = 5
            AverageBMSTemperature = 30.11
            GroupedCellsTemperatures = ListContainer:
                30.01
                30.01
                30.01
                29.91
            Current = 0.0
            Voltage = 52.965
            Power = 0.0
            RemainingCapacity = 65.535
            UserDefinedItems = 4
            TotalCapacity = 65.535
            CycleNumber = 41
            US3000 = Container:
                Remaining = 74.0
                Total = 74.0
        Container:
            NumberOfCells = 15
            CellVoltages = ListContainer:
                3.529
                3.528
                3.519
                3.533
                3.529
                3.532
                3.53
                3.532
                3.536
                3.536
                3.537
                3.536
                3.537
                3.533
                3.535
            NumberOfTemperatures = 5
            AverageBMSTemperature = 30.11
            GroupedCellsTemperatures = ListContainer:
                29.91
                29.91
                29.91
                29.91
            Current = 0.0
            Voltage = 52.982
            Power = 0.0
            RemainingCapacity = 65.535
            UserDefinedItems = 4
            TotalCapacity = 65.535
            CycleNumber = 114
            US3000 = Container:
                Remaining = 74.74
                Total = 74.0
        Container:
            NumberOfCells = 15
            CellVoltages = ListContainer:
                3.533
                3.525
                3.532
                3.53
                3.532
                3.522
                3.521
                3.524
                3.539
                3.539
                3.522
                3.537
                3.538
                3.54
                3.539
            NumberOfTemperatures = 5
            AverageBMSTemperature = 30.01
            GroupedCellsTemperatures = ListContainer:
                29.81
                29.81
                29.91
                29.81
            Current = 0.0
            Voltage = 52.973
            Power = 0.0
            RemainingCapacity = 65.535
            UserDefinedItems = 4
            TotalCapacity = 65.535
            CycleNumber = 108
            US3000 = Container:
                Remaining = 74.0
                Total = 74.0
    TotalPower = 0.0
    StateOfCharge = 1.0
>>>