-
Notifications
You must be signed in to change notification settings - Fork 38
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
Implent JointRegularizationTask in TSID #170
Conversation
ce0c3fe
to
a44e855
Compare
bool setReferenceTrajectory(Eigen::Ref<const Eigen::VectorXd> jointPosition); | ||
|
||
/** | ||
* Set the desired reference trajectory. | ||
* @param jointPosition vector containing the desired joint position in radians. | ||
* @param jointVelocity vector containing the desired joint velocity in radians per second. | ||
* @note The desired acceleration is implicitly set to zero. | ||
* @return True in case of success, false otherwise. | ||
*/ | ||
bool setReferenceTrajectory(Eigen::Ref<const Eigen::VectorXd> jointPosition, | ||
Eigen::Ref<const Eigen::VectorXd> jointVelocity); | ||
|
||
/** | ||
* Set the desired reference trajectory. | ||
* @param jointPosition vector containing the desired joint position in radians. | ||
* @param jointVelocity vector containing the desired joint velocity in radians per second. | ||
* @param jointAcceleration vector containing the desired joint velocity in radians per second square. | ||
* @return True in case of success, false otherwise. | ||
*/ | ||
bool setReferenceTrajectory(Eigen::Ref<const Eigen::VectorXd> jointPosition, | ||
Eigen::Ref<const Eigen::VectorXd> jointVelocity, | ||
Eigen::Ref<const Eigen::VectorXd> jointAcceleration); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are not setting trajectories, but rather desired set points.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed here: 40b8882
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to mess up with you a little more 😁 setSetpoints
sounds funny, setReferencePoint
? 😁
constexpr std::string_view descriptionPrefix = "JointRegularizationTask Optimal Control " | ||
"Element"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure this adds any detail in the error messages. Also, it is no more an Optimal Control Element
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove here: 2734000
- Remove useless printed messages from JointsTrackingTask
…ngTask::setSetpoint()
This PR:
Task::subA()
methodJointRegularizationTask
in TSID library