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

Cleanup in imu_pub #320

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

Cleanup in imu_pub #320

vooon opened this issue Jun 28, 2015 · 1 comment
Labels
Milestone

Comments

@vooon
Copy link
Member

vooon commented Jun 28, 2015

After switching to Eigen first place to clean-up is IMU plugin.
Most of vector items multiplications may be replaced with single vector * scalar operation (and some platform can accelerate that).

@vooon vooon added the plugin label Jun 28, 2015
@vooon vooon added this to the Version 0.13 milestone Jun 28, 2015
vooon added a commit that referenced this issue Jul 2, 2015
@vooon vooon closed this as completed in 5aa17c2 Jul 2, 2015
@vooon
Copy link
Member Author

vooon commented Jul 2, 2015

Tested by APM SITL. I hope that PX4 also works.

vooon added a commit that referenced this issue Jul 2, 2015
* master:
  plugin: imu_pub fix #320: move constants outside class, else runtime linkage error.
  plugin: imu_pub #320: first attempt
  eigen #319: handy wrappers.
  eigen #319: add euler-quat function.
  test #321: remove duplicated test cases, separate by library.
  test #321: testing eigen-based transforms.
  mavros #319: Add Eigen dependency and cmake rule.
  0.12.0
  Prepare release 0.12
  test: import launch for imu testing
  test: apm sitl and imu test reproduction steps
  test: test for UAS::transform_frame_attitude_rpy() (ERRORs!)
  test: test for UAS::transform_frame_xyz()
  test: Initial import test_frame_conv
  test: Add test_marvros package stub
@vooon vooon mentioned this issue Jul 21, 2015
4 tasks
nmichael pushed a commit to rislab/mavros that referenced this issue Mar 19, 2016
Add EKF_EXT and ASL_OBCTRL messages to ASLUAV
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant