Skip to content

Conversation

@MichaelGrupp
Copy link
Member

Previous behavior: if you dragged the URDF file into the viewer, it created its own new recording. Same for "Import into current recording" menu (note: requires #11941), which is counter-intuitive.

A recording containing only an URDF is pretty useless, and it's more common that you have some recording and want to add the model to it.

Loosely related to https://linear.app/rerun/issue/RR-2995

Previous behavior: if you dragged the URDF file into the viewer,
it created its own new recording. Same for "Import into current
recording" menu (note: requires #11941), which is counter-intuitive.

A recording containing only an URDF is pretty useless, and it's
more common that you have some recording and want to add the model
to it.
@github-actions
Copy link

github-actions bot commented Nov 21, 2025

Web viewer built successfully.

Result Commit Link Manifest
b6b0f8b https://rerun.io/viewer/pr/11944 +nightly +main

View image diff on kitdiff.

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

@MichaelGrupp MichaelGrupp changed the title Allow to import URDF file into current recording Load URDF file into the current recording, if one exists already Nov 21, 2025
@MichaelGrupp MichaelGrupp added ui concerns graphical user interface include in changelog feat-dataloader Everything related to data loaders labels Nov 21, 2025
@MichaelGrupp
Copy link
Member Author

cargo run -p animated_urdf that logs URDF via SDK works still fine with this (ran it with an open viewer that had already a recording, and the logging example created a new one as desired).

@MichaelGrupp MichaelGrupp merged commit 1013532 into main Nov 21, 2025
49 of 50 checks passed
@MichaelGrupp MichaelGrupp deleted the michael/urdf-into-current-recording branch November 21, 2025 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat-dataloader Everything related to data loaders include in changelog ui concerns graphical user interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants