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

[WIP] Update CameraBaseToOffset #1506

Draft
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

sktometometo
Copy link
Contributor

@sktometometo sktometometo commented Jun 12, 2022

This PR is integrated version of #1312 and #1504 and other features.

This PR add features to CameraBaseToOffset.py below.

Transform calculation is thanks to @iory @Naoki-Hiraoka @HiroIshida

Features

TODO

  • documentation に transform についての数式・図を追加する(Add t265 odometry transformer #1504 の図を参照)
  • covarianceの計算を修正。計算がちょっと多いのでon/offのparamがあるといい?
    • covariance は 決め打ち一定値、変換された値、そのままの3種類をparamで選択できるといい
  • docsの導出を修正、多分ちょこちょこ間違っている。とくに vの変換
  • odomの原点の座標系としてどのようなものをサポートするかリストアップ&paramで選択できるように
  • T265方式のOdometryをサポート(odometry origin が $\Sigma_\mathrm{Ovis}ではなく重力方向に補正されている。
  • T265でのワーキングサンプル他のvisual odomでのワーキングサンプル みたいなものの追加
  • 新しいロボットで使用する際の Tutorial の追加

@sktometometo
Copy link
Contributor Author

座標系の闇に飲まれつつあるので、 https://github.com/sktometometo/jsk_robot/blob/PR/udpate-camera-base-to-offset/jsk_robot_common/jsk_robot_startup/README.md の計算式・導出見てておかしいところ見つけた人がいればコメントもらえると助かります。

@sktometometo sktometometo force-pushed the PR/udpate-camera-base-to-offset branch from b54a803 to d132467 Compare June 13, 2022 04:02
@sktometometo
Copy link
Contributor Author

面倒になったんで暫く塩漬けします

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants