Skip to content

Conversation

@julieta-prieto
Copy link
Contributor

@julieta-prieto julieta-prieto commented Dec 29, 2025

Description

Canopen registers can be read using complete access right now.
However, the arguments that were used for __read_raw() method were incorrect, leading to exceptions:

image

Type of change

Please add a description and delete options that are not relevant.

  • Fixed read_complete_access read arguments for canopen registers.
  • Updated DriveContextManager to just read registers using complete access for EtherCAT servos.

Tests

  • Add new unit tests if it applies.

Please describe the manual tests that you ran to verify your changes if it applies.

Documentation

Please update the documentation.

  • Update docstrings of every function, method or class that change.
  • Build documentation locally to verify changes.
  • Add the changes at the [Unreleased] section of the CHANGELOG.

@julieta-prieto julieta-prieto merged commit 70234eb into develop Jan 8, 2026
1 check passed
@julieta-prieto julieta-prieto deleted the fix_complete_access_canopen branch January 8, 2026 08:28
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.

4 participants