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.

Sign up for GitHub

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

Jump to bottom

Eigen::Map<Quaternionf> place data in xyzw order instead of wxyz. #352

Closed
vooon opened this issue Jul 28, 2015 · 1 comment
Closed

Eigen::Map<Quaternionf> place data in xyzw order instead of wxyz. #352

vooon opened this issue Jul 28, 2015 · 1 comment
Assignees
@vooon
Labels
bug
Milestone
Version 0.13

Comments

@vooon
Copy link
Member

vooon commented Jul 28, 2015

Found in PR #351.
@vooon vooon added the bug label Jul 28, 2015
@vooon vooon self-assigned this Jul 28, 2015
@vooon vooon added this to the Version 0.13 milestone Jul 28, 2015
@vooon vooon mentioned this issue Jul 28, 2015
Closed
4 tasks
vooon added a commit that referenced this issue Jul 29, 2015
@vooon
lib #352: Add helper function UAS::quaternion_to_mavlink()
9990651
vooon added a commit that referenced this issue Jul 29, 2015
@vooon
Merge branch 'master' into coverity_scan
399d168 
* master:
  lib #352: Add helper function UAS::quaternion_to_mavlink()
  package: Add Eigen search path
  Fixed bug in send_attitude_target()
@vooon
Copy link
Member Author

vooon commented Jul 29, 2015

That issue is surprise to me, because at API level Eigen::Quaternion uses wxyz, so different internal representation is weird.

vooon added a commit that referenced this issue Jul 29, 2015
@vooon
plugin: setpoint_attitude #352: use new helper.
805112a
@vooon vooon closed this as completed in 231d77e Jul 29, 2015
vooon added a commit that referenced this issue Jul 29, 2015
@vooon
Merge branch 'master' into coverity_scan
924a8ce 
* master:
  extras: mocap fix #352: use new helper for quaternion.
  plugin: setpoint_attitude #352: use new helper.
  plugin: sys: Fix cppcheck and YouCompleteMe warnings
  plugin: ftp: Fix cppcheck errors.
nmichael pushed a commit to rislab/mavros that referenced this issue Mar 19, 2016
@LorenzMeier
Merge pull request mavlink#352 from matt-beall/namedUnnamedStruct
7a7ce33 
Named an unnamed struct
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vooon vooon

Labels
bug
Projects
None yet
Milestone
Version 0.13
Development

No branches or pull requests
1 participant
@vooon