Skip to content

[algorithm] Add AdvancedTimer calls for profiling in hot stages of the algorithm #99

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.

Sign up for GitHub

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

Jump to bottom
Merged
epernod merged 23 commits into from
Sep 26, 2025
Merged

[algorithm] Add AdvancedTimer calls for profiling in hot stages of the algorithm #99

epernod merged 23 commits into from
Sep 26, 2025

Conversation

@th-skam
Copy link
Collaborator

@th-skam th-skam commented Sep 23, 2025

Advanced Timer Additions

  • In InsertionAlgorithm::doDetection and individual algorithm steps
  • Enabled it in BaseAABBBroadPhase

Based on #98

th-skam and others added 13 commits September 20, 2025 00:26
@th-skam
[algorithm] Remove projection operation on tip in the InsertionAlgori…
dde0315 
…thm- completely unnecessary
@epernod
Merge branch 'master' into improve-projective
9b6039c
@th-skam
[algorithm] Changed FindClosestProximityAlgorithm to find the closest…
71782d1 
… proximity back on the from geometry
@th-skam
[algorithm] Auto-format code
69ded04
@th-skam
[algorithm] Rewrite some comments
38608bb
@th-skam
[algorithm] Place shared operations at the top
7241d39
@th-skam
[algorithm] Add a bool to enable/disable the puncture sequence
adf8d79
@th-skam
[algorithm] Add a bool to enable/disable insertion sequence
62b077a
@th-skam
[algorithm] Add a bool to enable/disable shaft collision sequence
4400991
@th-skam
[algorithm] auto format code
ec35524
@th-skam
[algorithm] Absorb the FindClosestProximity for shaft collision
91ad07d
@th-skam
[algorithm] Add an experimental component in the algorithm
f57b898
@th-skam
[algorithm] Remove FindClosestProximityAlgorithm
181dc54
@th-skam th-skam force-pushed the dev-profiling branch 2 times, most recently from fcfdf0c to 13974a5 Compare September 23, 2025 11:52
@th-skam th-skam added pr: status to review To notify reviewers to review this pull-request and removed pr: status wip labels Sep 24, 2025
@th-skam th-skam mentioned this pull request Sep 24, 2025
Merged
@epernod epernod added pr: status ready Approved a pull-request, ready to be squashed and removed pr: status to review To notify reviewers to review this pull-request labels Sep 26, 2025
@epernod epernod merged commit 640d11e into master Sep 26, 2025
4 checks passed
@epernod epernod deleted the dev-profiling branch September 26, 2025 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@epernod epernod epernod approved these changes

Assignees

No one assigned

Labels

pr: new feature pr: status ready Approved a pull-request, ready to be squashed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@th-skam @epernod