Skip to content

rs-ar-basic: fix extrinsic pose to camera transformation#4945

Merged
dorodnic merged 2 commits into
realsenseai:developmentfrom
radfordi:ar-basic-extrinsics
Oct 2, 2019
Merged

rs-ar-basic: fix extrinsic pose to camera transformation#4945
dorodnic merged 2 commits into
realsenseai:developmentfrom
radfordi:ar-basic-extrinsics

Conversation

@radfordi

Copy link
Copy Markdown
Contributor

Found by user @JBBee: #4883 (comment).

@dorian3d

Copy link
Copy Markdown
Contributor

@radfordi Could you also update this comment:

// This is the pose of the fisheye sensor relative to the T265 coordinate system.

It should say the opposite:

// This is the pose of the T265 coordinate system relative to the fisheye sensor

@radfordi

Copy link
Copy Markdown
Contributor Author

Thanks @dorian3d. I changed the variable name to replace the comment. That way it makes the usage site clearer as well.

@dorian3d

Copy link
Copy Markdown
Contributor

Thanks @radfordi. I forgot about the readme. Do you mind updating it too?

@dorodnic dorodnic added the tracking 6-DOF tracking, SLAM and T26x series label Oct 2, 2019
@dorodnic dorodnic changed the base branch from master to development October 2, 2019 08:09
@dorodnic dorodnic merged commit 3e745d9 into realsenseai:development Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tracking 6-DOF tracking, SLAM and T26x series

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants