Description
Hi I know this repo is no longer active development. However I am unclear whether that means others can not open pull requests an have them merged? Or just that the main developers are no longer actively contributing code. If desired I can happily open a pull request to fix this issue.
In any case I have found a bug with the decode method for the CIP array type in the current version
Specifically this line:
|
_val = [cls.element_type.decode(stream) for _ in range(_length)] |
references _length when it should be using _len which results in the error
"TypeError: '_DataTypeMeta' object cannot be interpreted as an integer" when trying to decode an array specified in a manner such as
Array(UINT, ActiveLanguage)
Description
Hi I know this repo is no longer active development. However I am unclear whether that means others can not open pull requests an have them merged? Or just that the main developers are no longer actively contributing code. If desired I can happily open a pull request to fix this issue.
In any case I have found a bug with the decode method for the CIP array type in the current version
Specifically this line:
pycomm3/pycomm3/cip/data_types.py
Line 872 in 36aeb19
references _length when it should be using _len which results in the error
"TypeError: '_DataTypeMeta' object cannot be interpreted as an integer" when trying to decode an array specified in a manner such as
Array(UINT, ActiveLanguage)