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

Ordering of the extrinsic rotation #2543

Closed
geoeo opened this issue Nov 14, 2022 · 5 comments
Closed

Ordering of the extrinsic rotation #2543

geoeo opened this issue Nov 14, 2022 · 5 comments
Assignees
Labels

Comments

@geoeo
Copy link

geoeo commented Nov 14, 2022

The rotation in the extrinsic message is defined as float[9].
But it does not state explicitly if its row major or column major.

Which one does the array in the extrinsic message represent?

Best,
Marc

@MartyG-RealSense
Copy link
Collaborator

MartyG-RealSense commented Nov 14, 2022

Hi @geoeo The official SDK documentation page link below states extrinsic rotation to be column major.

https://intelrealsense.github.io/librealsense/python_docs/_generated/pyrealsense2.extrinsics.html

@geoeo
Copy link
Author

geoeo commented Nov 14, 2022

Thanks @MartyG-RealSense !

Thats a pretty obscure link. A mention of this in the Readme.md would be appreciated!

@MartyG-RealSense
Copy link
Collaborator

MartyG-RealSense commented Nov 16, 2022

I will add a Enhancement label to this case to highlight it in the future to Intel RealSense colleagues responsible for documentation. This case should be kept open but you do not need to take any further action. Thanks again!

@SamerKhshiboun
Copy link
Contributor

@MartyG-RealSense, the fix #2693 was merged.
You can close this ticket

@MartyG-RealSense
Copy link
Collaborator

Thanks very much @SamerKhshiboun :)

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

3 participants