search

christiansandberg / canopen

CANopen for Python
http://canopen.readthedocs.io/
MIT License
449 stars 197 forks source link

Index + Subindex missing from abort messages #377

Open mbs38 opened 1 year ago

mbs38 commented 1 year ago

Hi, thanks for the great work.

I noticed that index + subindex are missing from the client side abort messages when comparing the raw data to a commercial tool I'm using. Index + subindex have to be included according to the standard. See https://forum.opencyphal.org/uploads/short-url/mNWuvY23DYckSFoSbfam0ji3YWn.pdf Page 61.

Code: https://github.com/christiansandberg/canopen/blob/e8807b87e69902840f459279ce38e330b069f53d/canopen/sdo/client.py#LL92C1-L100C1

Thx!

friederschueler commented 5 months ago

I think this is a bug, because we are not following the standard.

Could you provide the different raw data?