rerun-io / rerun

Visualize streams of multimodal data. Free, fast, easy to use, and simple to integrate. Built in Rust.
https://rerun.io/
Apache License 2.0
6.68k stars 338 forks source link

Fix `Loggable` pretending to have semantics #8082

Closed teh-cmc closed 1 week ago

teh-cmc commented 2 weeks ago

Remove Loggable's associated Name type and simplify the Loggable and Component types accordingly.

A Loggable doesn't have a name, in fact it doesn't have any semantics at all: it's just arrow-serializable data. Component is where semantics live at.

Not only is this wrong, this adds a whole bunch of useless code that makes future improvements (hints: tags) more complicated for no reason.

teh-cmc commented 2 weeks ago

@rerun-bot full-check

github-actions[bot] commented 2 weeks ago

Started a full build: https://github.com/rerun-io/rerun/actions/runs/11784048498