Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(NAV-1380) Feature: velocity smoother #23

Merged
merged 4 commits into from
Jan 2, 2023
Merged

Conversation

arekin19
Copy link

@arekin19 arekin19 commented Dec 29, 2022

Purpose

Serena's movements are not smooth. Especially after re-planning around obstacles, trajectory changes and robot is doing jerky moves.

Approach

ros-planning/navigation2 on main introduced package velocity_smoother: link. We are cherry-picking it with small adaptation to make it work with old version of Life-cycle Node. Added remapping allows to use it with multiplexer input.

"The aim of this package is to implement velocity, acceleration, and dead band smoothing from Nav2 to reduce wear-and-tear on robot motors and hardware controllers by smoothing out the accelerations/jerky movements that might be present with some local trajectory planners' control efforts." NAV2 documentation

Package is scaling incoming velocity commands to keep accelerations and deceleration within given bounds and maintain the trajectory. All components of velocity are scaled proportionally to a component that require changes due to acceleration limits. Package supports dynamically reconfigurable parameters.

Testing

Results described here: serena_bringup/109

Open Questions and Pre-Merge TODOs

n/a

Dependencies

Disclaimers

n/a

Learning

Additional documentation

System Architecture, section 1.1.4.2 Velocity Smoother

Blog Posts

@arekin19 arekin19 marked this pull request as ready for review December 30, 2022 11:17
@arekin19 arekin19 merged commit 81a93e8 into galactic Jan 2, 2023
@arekin19 arekin19 deleted the sync/velocity-smoother branch January 2, 2023 15:52
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.

3 participants