Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

send_column for encoded image does not assign an encoded image visualizer #7980

Closed
oceanusxiv opened this issue Nov 4, 2024 · 2 comments
Closed
Labels
😤 annoying Something in the UI / SDK is annoying to use ❓ question Further information is requested

Comments

@oceanusxiv
Copy link

Describe the annoyance

To Reproduce
Steps to reproduce the behavior:

  1. run, where encoded_images is a list of binary blobs
rr.send_columns(
    "observation/wrist_image_left",
    times=[rr.TimeSequenceColumn("step", timesteps)],
    components=[
        rr.components.BlobBatch(data=encoded_images)
    ],
)
  1. by default a spatial2Dview will not register a visualizer for this blob, even if I specify a MediaType column to specify the MIME type.

Expected behavior
Ideally if you pass the MIME type along, or really even if you didn't, the viewer should probably be able to guess a good visualizer for it, like other send_column types.

Your goals

Screenshots

Desktop (please complete the following information):

  • OS: macOS Sonoma 14.6

Additional context

@oceanusxiv oceanusxiv added 👀 needs triage This issue needs to be triaged by the Rerun team 😤 annoying Something in the UI / SDK is annoying to use labels Nov 4, 2024
@Wumpf
Copy link
Member

Wumpf commented Nov 4, 2024

What makes the visualizer show up automatically is the indicator component, not the media type. This is why the example snippet for images first does a regular log first since that's the easiest way to get that in (this applies for encoded images just like everything else).

This is a bit of an arcane corner of Rerun and not super well documented (it is mentioned on https://rerun.io/docs/concepts/visualizers-and-overrides#how-are-visualizations-produced but easy to miss). However, we want to remove those indicators and bake them into individual component tags, see


Overall definitely an annoying issue, very much agreed! But since it's technically documented, works as designed and should go away I'm closing this as a duplicate of tagged components

@Wumpf Wumpf closed this as not planned Won't fix, can't repro, duplicate, stale Nov 4, 2024
@Wumpf Wumpf added ❓ question Further information is requested and removed 👀 needs triage This issue needs to be triaged by the Rerun team labels Nov 4, 2024
@oceanusxiv
Copy link
Author

Oh that's how you do it! Thanks for the tip.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
😤 annoying Something in the UI / SDK is annoying to use ❓ question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants