Error in node.pdo.save()

Hello.

I got the error messages as below:

Traceback (most recent call last):
  File "testing.py", line 55, in <module>
    motor_node.tpdo.save()
  File "/usr/local/lib/python3.7/site-packages/canopen/pdo/base.py", line 57, in save
    pdo_map.save()
  File "/usr/local/lib/python3.7/site-packages/canopen/pdo/base.py", line 334, in save
    self.com_record[1].raw = self.cob_id | PDO_NOT_VALID
TypeError: unsupported operand type(s) for |: 'NoneType' and 'int'

with the codes like below:

      node.tpdo[1].clear()
      node.tpdo[1].add_variable('8 bit digital input block area', '1. digital input block')
      node.tpdo[1].add_variable('8 bit digital input block area', '2. digital input block')
      node.tpdo[1].enabled = True
      node.tpdo.save()

I have checked some variables, node.tpdo[1].cob_id is None, and node.tpdo[1].predefined_cob_id is 0x181 (my node id is 1).

Did I misunderstand something or should something be improved? Thanks very much.

[zcchen]

Sorry for my late reply. I think we can assume this topic as closed.

Most devices have EDS files usually for the product version. But as a R&D engineer, I have to touch many under-developing devices which do NOT have any EDS files. Meanwhile, some device suppliers in China provide incorrect EDS files but correct manuals for CANopen dictionaries. Thus, we have to avoid any EDS files in our production codes.

That was why I would like to have this feature for some spike testing. If we have this feature in future, that would be great. If no, there are also OK since I have implemented a workaround solution. If I have time, I can try to contribute my codes to this repo.

Thanks very much for everyone.

[af-silva]

Ok, you are welcome @zcchen to contribute. Keep hacking! xD