Skip to content

Bump 1.3.0#98

Merged
robotpilot merged 9 commits intojazzyfrom
main
May 6, 2026
Merged

Bump 1.3.0#98
robotpilot merged 9 commits intojazzyfrom
main

Conversation

@sunghowoo
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Sungho Woo <wsh@robotis.com>
Signed-off-by: Sungho Woo <wsh@robotis.com>
Signed-off-by: Sungho Woo <wsh@robotis.com>
Signed-off-by: Sungho Woo <wsh@robotis.com>
Signed-off-by: Sungho Woo <wsh@robotis.com>
Signed-off-by: Sungho Woo <wsh@robotis.com>
Signed-off-by: Sungho Woo <wsh@robotis.com>
Improved 3-wheel swerve steering limit using flip logic
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the package version to 1.3.0 across multiple packages and includes significant improvements to the swerve drive controller, specifically implementing a smooth 180-degree steering reversal logic and optimizing the real-time update loop. The review identified several critical issues, including unsafe usage of std::optional::value() which could cause runtime exceptions, a typo in a member variable name, and an unnecessary bias introduced in the atan2 calculation. The reviewer also suggested improvements for handling hardware state reading failures to ensure robustness in the real-time control loop.

Comment thread ffw_swerve_drive_controller/src/swerve_drive_controller.cpp
Comment thread ffw_swerve_drive_controller/src/swerve_drive_controller.cpp
Comment thread ffw_swerve_drive_controller/src/swerve_drive_controller.cpp
Comment thread ffw_swerve_drive_controller/src/swerve_drive_controller.cpp
Comment thread ffw_swerve_drive_controller/src/swerve_drive_controller.cpp
@sunghowoo sunghowoo added the bump Increase the version number to release label May 6, 2026
@robotpilot robotpilot merged commit abe5ddc into jazzy May 6, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bump Increase the version number to release

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants