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

express the accelerations in the body frame #16

Merged
merged 1 commit into from
Jan 15, 2019

Conversation

doudou
Copy link
Contributor

@doudou doudou commented Dec 13, 2018

This is really what makes the most sense from a sensing perspective.
Sensors (gyros, accelerometers) and models (hydrodynamic models)
give access to velocities and accelerations w.r.t. the inertial
frame, but expressed in the body frame.

However, expressing the velocities/accelerations in the world frame
from these sensing/estimation methods would require to have an estimate
of the system's pose in the world, which is a harder thing to get

@doudou
Copy link
Contributor Author

doudou commented Dec 13, 2018

@alexfneves hopefully the last PR on the subject

This is really what makes the most sense from a sensing perspective.
Sensors (gyros, accelerometers) and models (hydrodynamic models)
give access to velocities and accelerations w.r.t. the inertial
frame, but expressed in the body frame.

However, expressing the velocities/accelerations in the world frame
from these sensing/estimation methods would require to have an estimate
of the system's pose in the world, which is a harder thing to get
@doudou doudou force-pushed the express_accelerations_in_the_body_frame branch from aead319 to 380817a Compare December 13, 2018 19:36
@alexfneves
Copy link
Contributor

alexfneves commented Dec 19, 2018

The linear acceleration is now written on the body frame when a link is exported

@doudou
Copy link
Contributor Author

doudou commented Dec 19, 2018

The linear acceleration is now written on the body frame when a link is exported

Yes. Which is the (now) expected behavior per the PR message and rock-core/base-types#124

@doudou doudou merged commit 5ca3fa8 into master Jan 15, 2019
@doudou doudou deleted the express_accelerations_in_the_body_frame branch January 15, 2019 15:02
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