-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[geometry] Avoid slowdown of SceneGraph::GetDirectFeedthroughs() #21630
[geometry] Avoid slowdown of SceneGraph::GetDirectFeedthroughs() #21630
Conversation
Use an explicit list of dependency tickets in SceneGraph to avoid a pile of useless reifications of proximity geometry. The useless work is triggered if the base class LeafSystem::GetDirectFeedthroughs() implementation falls back on its technique of making a SystemSymbolicInspector, which needs to build an Expression-typed SceneGraph context.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Closes: #21624.
+(release notes: fix) (don't do insanely slow waste work)
+@joemasterjohn for feature review.
Reviewable status: LGTM missing from assignee joemasterjohn, needs platform reviewer assigned, needs at least two assigned reviewers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
feature. +@EricCousineau-TRI for platform review, please.
Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: LGTM missing from assignee EricCousineau-TRI(platform)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: complete! all discussions resolved, LGTM from assignees EricCousineau-TRI(platform),joemasterjohn
…otLocomotion#21630) Use an explicit list of dependency tickets in SceneGraph to avoid a pile of useless reifications of proximity geometry. The useless work is triggered if the base class LeafSystem::GetDirectFeedthroughs() implementation falls back on its technique of making a SystemSymbolicInspector, which needs to build an Expression-typed SceneGraph context.
Use an explicit list of dependency tickets in SceneGraph to avoid a pile of useless reifications of proximity geometry. The useless work is triggered if the base class LeafSystem::GetDirectFeedthroughs() implementation falls back on its technique of making a SystemSymbolicInspector, which needs to build an Expression-typed SceneGraph context.
This change is