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

Should IMU config Tw and R_IMUtoGYRO both be set to Kalibr imu0:gyroscope:M? #358

Closed
cod3monk3y opened this issue Jul 19, 2023 · 1 comment · Fixed by #395
Closed

Should IMU config Tw and R_IMUtoGYRO both be set to Kalibr imu0:gyroscope:M? #358

cod3monk3y opened this issue Jul 19, 2023 · 1 comment · Fixed by #395
Labels
bug Something isn't working

Comments

@cod3monk3y
Copy link

Just clarifying comments from kalibr_imu_chain.yaml in config/*/kalibr_imu_chain.yaml. I checked (almost) all of the configs and they all have the same comment (minor indentation added for clarity):

  # how to get from Kalibr imu.yaml result file:
  #   - Tw is           imu0:gyroscopes:M:
  #   - R_IMUtoGYRO: is imu0:gyroscopes:M:
  #   - Ta is           imu0:accelerometers:M:
  #   - R_IMUtoACC      not used by Kalibr
  #   - Tg is           imu0:gyroscopes:A:

Is it correct that Tw and R_IMUtoGYRO should both be imu0:gyroscopes:M:? If so, this leaves imu0:gyroscopes:C_gyro_i from Kalibr unused.

@goldbattle goldbattle added the bug Something isn't working label Jul 19, 2023
@goldbattle
Copy link
Member

No it would be C_gyro_i, good catch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants