Skip to content

Adds yaw offset property - #6

Closed
Rauldg wants to merge 1 commit into
rock-slam:masterfrom
Rauldg:add_yaw_offset
Closed

Adds yaw offset property#6
Rauldg wants to merge 1 commit into
rock-slam:masterfrom
Rauldg:add_yaw_offset

Conversation

@Rauldg

@Rauldg Rauldg commented Jan 26, 2024

Copy link
Copy Markdown

since the init pose property ignores the orientation part of the pose

@chhtz

chhtz commented Feb 2, 2024

Copy link
Copy Markdown
Member

I would kind-of prefer fixing the start_pose functionality. E.g., just always output start_pose * odo_pose (where odo_pose is what is currently calculated when start_pose is not set). But then renaming start_pose to pose_offset would probably make more sense. And it would be something which would also be achievable with the transformer (probably not as convenient as with a property).

I'll ask some other SLAM users for their opinion.

@planthaber

Copy link
Copy Markdown
Member

#7

@Rauldg Rauldg closed this Feb 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants