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

Allow the unicycle to move sideways #54

Merged
merged 8 commits into from
Sep 9, 2022
Merged

Allow the unicycle to move sideways #54

merged 8 commits into from
Sep 9, 2022

Conversation

S-Dafarra
Copy link
Collaborator

This PR adds a new controller for the Unicycle, permitting side steps.

Copy link
Member

@GiulioRomualdi GiulioRomualdi left a comment

Choose a reason for hiding this comment

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

One minor comment


#include "UnicycleBaseController.h"

class UnicycleDirectController : public UnicycleBaseController
Copy link
Member

Choose a reason for hiding this comment

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

If I correctly understand this allows the robot to move in the y direction (i.e. it brakes the holonomic constraint) if this is the case I would avoid calling it UnicycleDirectController

Copy link
Collaborator Author

@S-Dafarra S-Dafarra Sep 9, 2022

Choose a reason for hiding this comment

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

Nope, this is one unicycle controller that allows the user to set the unicycle velocities directly. The other controller is the PersonFollowing controller, where the user sets a desired point to follow.

In order to move sideways I have extended the dynamics of the unicycle in

stateDynamics(0) = c_theta * m_controllerOutput(0) - s_theta * m_controllerOutput(2);
stateDynamics(1) = s_theta * m_controllerOutput(0) + c_theta * m_controllerOutput(2);
, assuming to have an additional control input. The PersonFollowing controller simply sets this control input always to zero.

@GiulioRomualdi
Copy link
Member

Ok! Perfect feel free to merge it. Could you please release the software? It will allow us to release the walking as well.

@S-Dafarra
Copy link
Collaborator Author

Ok! Perfect feel free to merge it. Could you please release the software? It will allow us to release the walking as well.

Sure!

@S-Dafarra S-Dafarra merged commit 55852da into master Sep 9, 2022
@S-Dafarra S-Dafarra deleted the sidesteps branch September 9, 2022 14:57
@S-Dafarra
Copy link
Collaborator Author

Ok! Perfect feel free to merge it. Could you please release the software? It will allow us to release the walking as well.

See https://github.com/robotology/unicycle-footstep-planner/releases/tag/v0.5.0

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