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

Velocity Controller-Relax Momentum Constraint #107

Conversation

CarlottaSartore
Copy link
Contributor

@CarlottaSartore CarlottaSartore commented Feb 10, 2021

This PR implements the hyperbolic tangent method to constraint the CoM position and removes the Linear Momentum Constraint
i.e.

$$ \dot{x}_{CoM,Lower}tanh(\epsilon_l[x_{CoM}-x_{lower}])<J_{CMM}\dot{s}_{K+1}<tanh(\epsilon_u [x_{upper}- x_{CoM}])\dot{x}_{CoM,upper} $$

where

  • $x_{lower}$ is the lower bound of the support polygon;
  • $x_{upper}$ is the upper bound of the support polygon;
  • $x_{CoM}$ it the CoM Position computed via forward kinematic starting from $s$;
  • $\dot{x}_{CoM,Lower}$ is the lower bound for the CoM Velocity;
  • $\dot{x}_{CoM,Upper}$ is the upper bound for the CoM Velocity

In addition, minor fixes are introduced

C.C. @gabrielenava

@gabrielenava gabrielenava self-requested a review February 11, 2021 15:24
@gabrielenava
Copy link
Collaborator

I reviewed the code and it seemed ok to me at a first glance. I did not run any controller which uses this library, therefore I could not debug for theoretical issues. But we're currently testing this controller offline with @CarlottaSartore, and it seems working fine. Therefore merging.

@gabrielenava gabrielenava merged commit 0c7827f into robotology:devel Feb 11, 2021
gabrielenava pushed a commit that referenced this pull request Mar 26, 2021
* added a new momentum-velocity controller

* cleanup of the WCB simulink library

* improvements in velocity controller (#104)

* improvements in velocity controller

* added new line at the end of the files

* Velocity Controller-Relax Momentum Constraint  (#107)

* fixes centroidalTransformaion and ComputeReferencesController

* fixed some typos, removed hardcoded variables, clean up

* fixed indeces for velocity limits

* minor fixes

Co-authored-by: Giuseppe <gl.giuseppelerario@gmail.com>
Co-authored-by: CarlottaSartore <56030908+CarlottaSartore@users.noreply.github.com>
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