Skip to content

Conversation

@MichaelGrupp
Copy link
Member

@MichaelGrupp MichaelGrupp commented Nov 27, 2025

  • add CoordinateFrames to link entities and visual/collision meshes, attach meshes via Transform3D to their link
  • log (default) joints as Transform3D, with {parent, child}_frame connecting a pair of links

Note:

  • default transform values are static, but we want to be able to change them non-statically (https://linear.app/rerun/issue/RR-3042)
  • In order to see the model, there needs to be a connection to the view's root TF frame - this is planned to become a view property soon. Right now you probably need to change it in the selection panel of the root entity to e.g. base_link.
  • ❓Backwards compatibility with entity-transforms without frame ID: e.g. how the animated_urdf worked until now... doesn't seem to be compatible with this change.

Documentation will come separate (#12056)

@MichaelGrupp MichaelGrupp changed the title Load URDF with frame_ids Load URDF with frame IDs Nov 27, 2025
@github-actions
Copy link

github-actions bot commented Nov 27, 2025

Web viewer built successfully.

Result Commit Link Manifest
60cf31a https://rerun.io/viewer/pr/12005 +nightly +main

View image diff on kitdiff.

Note: This comment is updated whenever you push a commit.

@MichaelGrupp MichaelGrupp marked this pull request as draft November 27, 2025 14:19
@MichaelGrupp MichaelGrupp added do-not-merge Do not merge this PR feat-dataloader Everything related to data loaders labels Nov 27, 2025
@MichaelGrupp MichaelGrupp force-pushed the michael/urdf-tf-frame-support branch from c6b5316 to 3f8f272 Compare November 28, 2025 15:45
@MichaelGrupp MichaelGrupp force-pushed the michael/urdf-tf-frame-support branch from 3f8f272 to 9d62a33 Compare December 2, 2025 08:33
@MichaelGrupp MichaelGrupp added include in changelog and removed do-not-merge Do not merge this PR labels Dec 2, 2025
@MichaelGrupp MichaelGrupp marked this pull request as ready for review December 2, 2025 08:34
@MichaelGrupp MichaelGrupp added the 🪵 Log & send APIs Affects the user-facing API for all languages label Dec 2, 2025
@MichaelGrupp MichaelGrupp added the do-not-merge Do not merge this PR label Dec 2, 2025
// In order to move a joint, we just need to log a new transform between two of those frames.
rec.log(
link_path,
"/transforms",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is currently not working if the URDF is logged together with the transforms. Waiting for "static-like" TF that can be updated.

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

Labels

do-not-merge Do not merge this PR feat-dataloader Everything related to data loaders include in changelog 🪵 Log & send APIs Affects the user-facing API for all languages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants