Skip to content

PDO fixes #635

Closed
surajSFDC wants to merge 2 commits intocanopen-python:masterfrom
surajSFDC:fix/pdo-613-combined-maps-clean
Closed

PDO fixes #635
surajSFDC wants to merge 2 commits intocanopen-python:masterfrom
surajSFDC:fix/pdo-613-combined-maps-clean

Conversation

@surajSFDC
Copy link

  • Add PdoMaps map_offset/com_offset and getitem fallbacks for record indices
  • Replace dict merge with _CombinedPdoMaps; route 0x1400-0x17FF/0x1800-0x1BFF in PDO
  • Expand PdoBase fast path to 0x1400-0x1BFF; fix CiA 301 mapping tests (0x1A00 TPDO)
  • Add test_pdo_iterate

Fixes flawed combined-PDO approach from PR #613 (dummy zero offsets).

- Add PdoMaps map_offset/com_offset and __getitem__ fallbacks for record indices
- Replace dict merge with _CombinedPdoMaps; route 0x1400-0x17FF/0x1800-0x1BFF in PDO
- Expand PdoBase fast path to 0x1400-0x1BFF; fix CiA 301 mapping tests (0x1A00 TPDO)
- Add test_pdo_iterate

Fixes flawed combined-PDO approach from PR canopen-python#613 (dummy zero offsets).
@acolomb acolomb closed this Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants