Skip to content

perf(ViewNode): find direct children in getViewNodeFor - #3673

Merged
finetjul merged 1 commit into
Kitware:masterfrom
daker:perf-scenegraph
Oct 21, 2026
Merged

finetjul merged 1 commit into
Kitware:masterfrom
daker:perf-scenegraph

Conversation

@daker

@daker daker commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

Context

Results

Changes

getViewNodeFor looks in the renderable to child map before it searches the children recursively, so a direct child is found without a walk of the tree.

PR Checklist

  • GitHub Actions CI passed: semantic-release commit messages, lint, and tests
  • Test coverage added
  • Documentation and TypeScript definitions are updated to match these changes

getViewNodeFor looks in the renderable to child map before it searches
the children recursively, so a direct child is found without a walk
of the tree.
@daker

daker commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator Author

@PaulHax @finetjul FYI

@finetjul finetjul left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@daker

daker commented Oct 21, 2026

Copy link
Copy Markdown
Collaborator Author

@finetjul mind merging this one if it's ok ?

@finetjul
finetjul added this pull request to the merge queue Oct 21, 2026
Merged via the queue into Kitware:master with commit bc19b89 Oct 21, 2026
4 checks passed
@daker
daker deleted the perf-scenegraph branch October 21, 2026 11:07
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 37.4.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions Bot added the released Automated label label Oct 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Automated label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants