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

controller specific parseWheelTransform #196

Merged
merged 1 commit into from Nov 28, 2018

Conversation

Projects
None yet
1 participant
@fmessmer
Copy link
Member

commented Nov 16, 2018

works-around/fixes mojin-robotics/cob4#983

#195 is not the fix for mojin-robotics/cob4#983
but reverting (like #194) is not an option either

thus, I split the param_parsing again....but I'm still investigating...

@fmessmer

This comment has been minimized.

Copy link
Member Author

commented Nov 16, 2018

in addition, I found that driving backwards cannot be simulated in gazebo (see ipa320/cob_common#261)

@fmessmer

This comment has been minimized.

Copy link
Member Author

commented Nov 28, 2018

I can't figure out a parseWheelTransform computation that works for both - omni and tricycle, forward and backward - thus, I give each controller it's own parseWheelTransform function again

@fmessmer fmessmer changed the title [WIP] split parseWheelTransform controller specific parseWheelTransform Nov 28, 2018

@fmessmer fmessmer merged commit 4faac4e into ipa320:kinetic_dev Nov 28, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@fmessmer fmessmer deleted the fmessmer:split_parseWheelTransform branch Nov 28, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.