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

Documentation for Graceful Motion #517

Merged

Conversation

ajtudela
Copy link
Contributor

@ajtudela ajtudela commented Feb 5, 2024

Documentation for #4021

@SteveMacenski SteveMacenski merged commit 1ca7ce2 into ros-navigation:master Feb 5, 2024
1 check passed
@SteveMacenski
Copy link
Member

@ajtudela thanks for the contribution! If you had a video of this controller in action, that would be great for an announcement post to the community about your new controller plugin! Its worth spending a few minutes to make a nice demo video (then, that could be added to the config guide page as an example!)

@ajtudela
Copy link
Contributor Author

ajtudela commented Feb 6, 2024

I totally forgot! I made this video

I don't know if it's worth it.

@SteveMacenski
Copy link
Member

That video is a bit slow and glitchy (lots of jumps), if you could make another one at more reasonable speeds and smoothness, that would be great!

@ajtudela
Copy link
Contributor Author

ajtudela commented Feb 13, 2024

I haven't forgotten the video. It seems that YTdecrese the bitrate too much. So, I'd try to make a new video during this week.

@ajtudela
Copy link
Contributor Author

Next attempt! https://youtu.be/QHdrfadFhRg

@SteveMacenski
Copy link
Member

It seems a bit wobbly when there are small deviations in the path that the control point jitters between. Is there a nice way to smooth that out? (@mikeferguson perhaps a nice challenge when you're working on the dynamic point setting)

It definitely seems like keeping the point further off is a good move.

@ajtudela ajtudela deleted the feature/graceful_controller branch November 27, 2024 11:47
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