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

WIP: disconnect spaces when their categories differ #3790

Closed
wants to merge 1 commit into from

Conversation

jleibs
Copy link
Member

@jleibs jleibs commented Oct 10, 2023

What

Proof-of-concept mechanism to handle cases like: #3712

The basic idea here is to categorized space infos and treat them as disconnected when their categorization differs. Hits way too many edge-cases but useful to consider for proper solution.

Checklist

  • I have read and agree to Contributor Guide and the Code of Conduct
  • I've included a screenshot or gif (if applicable)
  • I have tested demo.rerun.io (if applicable)
  • The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG

@jleibs jleibs added the do-not-merge Do not merge this PR label Oct 10, 2023
@jleibs jleibs force-pushed the jleibs/space_info_categories branch from 2c258ab to 12cd512 Compare October 10, 2023 22:28
}
}

// TODO(jleibs): These need to get pulled from a registry of some form
Copy link
Member

@Wumpf Wumpf Oct 11, 2023

Choose a reason for hiding this comment

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

also ideally they're not in re_viewport but instead have this be governed by the heuristic method on the SpaceView trait. re_viewport knowing about re_types is kind of a hack

Copy link
Member

Choose a reason for hiding this comment

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

Associating some kind of "Preferred SpaceInfoCategory" metadata in the schema registry is an interesting thought.
Related:

Comment on lines +174 to +175
// TODO(jleibs): Why not make everything interesting?
return true;
Copy link
Member

@Wumpf Wumpf Oct 11, 2023

Choose a reason for hiding this comment

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

Much preferred. The whole idea of not having space_view at root be always a thing is from back then when we never allowed it in the first place.

The heuristics today tries to allow two ways of thinking and a mix of it right now: One where you have one scene at the root and one where you have many scenes each under a /thing. Back in the day we only had the latter.

@jleibs jleibs closed this Oct 23, 2023
@jleibs jleibs deleted the jleibs/space_info_categories branch June 14, 2024 13:02
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants