Skip to content

Conversation

quagla
Copy link
Collaborator

@quagla quagla commented Sep 3, 2025

With wp.Volume

Total simulation time: 180.79 s
Total steps per second: 45,312
Total realtime factor: 90.62 x
Total time per step: 22069.03 ns
Total converged worlds: 8192 / 8192

New implementation:

Total simulation time: 71.40 s
Total steps per second: 114,737
Total realtime factor: 229.47 x
Total time per step: 8715.59 ns
Total converged worlds: 8192 / 8192

Note that SDF collisions are not parallelized over the initial points yet (WIP #665)

@quagla quagla marked this pull request as ready for review September 3, 2025 11:39
@quagla quagla requested a review from thowell September 3, 2025 11:39
@thowell thowell merged commit 9e66324 into google-deepmind:main Sep 3, 2025
16 checks passed
@adenzler-nvidia
Copy link
Collaborator

not sure how this passed CI, but for me test_sdf_collision complains about missing volume_ids. Did I do a bad merge or is this change incomplete?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants