Skip to content

Update connector for move_go2_action mode#2066

Merged
openminddev merged 2 commits intomainfrom
fix-config
Feb 2, 2026
Merged

Update connector for move_go2_action mode#2066
openminddev merged 2 commits intomainfrom
fix-config

Conversation

@openminddev
Copy link
Copy Markdown
Contributor

This pull request makes a minor update to the config/unitree_go2_modes.json5 configuration file, changing the connector for the move_go2_action mode to use the more general unitree_sdk instead of unitree_rplidar_sdk.

  • Updated the connector for the move_go2_action mode from unitree_rplidar_sdk to unitree_sdk in config/unitree_go2_modes.json5

Changed the connector for the move_go2_action mode from 'unitree_rplidar_sdk' to 'unitree_sdk' in the unitree_go2_modes.json5 configuration file.
@openminddev openminddev requested review from a team as code owners February 2, 2026 20:18
Copilot AI review requested due to automatic review settings February 2, 2026 20:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the connector configuration for the move_go2_action mode to use the more general unitree_sdk instead of the specialized unitree_rplidar_sdk.

Changes:

  • Modified the connector property for move_go2_action mode from unitree_rplidar_sdk to unitree_sdk

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Introduces a 'version' attribute to ModeSystemConfig, updates related functions to handle the new field, and adds tests to ensure the version is included in serialization and object creation.
@openminddev openminddev requested a review from a team as a code owner February 2, 2026 20:30
@openminddev openminddev merged commit 5083565 into main Feb 2, 2026
1 of 6 checks passed
@openminddev openminddev deleted the fix-config branch February 2, 2026 20:33
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