Skip to content

Use full component descriptors optionally in ranges queries #9883

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Wumpf
Copy link
Member

@Wumpf Wumpf commented May 5, 2025

Related

What

Uses the same temporary MaybeTagged construct for range queries and makes range query results being indexed over tagged components

@Wumpf Wumpf added 🚜 refactor Change the code, not the functionality exclude from changelog PRs with this won't show up in CHANGELOG.md labels May 5, 2025
Copy link

github-actions bot commented May 5, 2025

Web viewer built successfully. If applicable, you should also test it:

  • I have tested the web viewer
Result Commit Link Manifest
5a51331 https://rerun.io/viewer/pr/9883 +nightly +main

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

@Wumpf Wumpf changed the title Make ComponentPath use full descriptor Use full component descriptors in range query results May 5, 2025
@Wumpf Wumpf changed the title Use full component descriptors in range query results Use full component descriptors optionally in ranges queries May 5, 2025
@Wumpf Wumpf requested a review from grtlr May 5, 2025 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exclude from changelog PRs with this won't show up in CHANGELOG.md 🚜 refactor Change the code, not the functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant