Skip to content

Use the getRawSkeleton() function for better performance#2

Open
mxgrey wants to merge 14 commits intogazebo-forks:azeey/friction_per_shape_more_paramsfrom
dartsim:grey/use_raw_skeleton_ptr
Open

Use the getRawSkeleton() function for better performance#2
mxgrey wants to merge 14 commits intogazebo-forks:azeey/friction_per_shape_more_paramsfrom
dartsim:grey/use_raw_skeleton_ptr

Conversation

@mxgrey
Copy link

@mxgrey mxgrey commented Jul 23, 2019

On dart's speed_test example application, this improved performance by ~12% on my machine.

@azeey
Copy link
Collaborator

azeey commented Jul 24, 2019

I tested this with one of the SubT practice tunnel circuits. Unfortunately I didn't see a noticeable performance improvement. This is probably because the time spent in step() is dominated by collision checking for this particular world.

@azeey
Copy link
Collaborator

azeey commented Jul 29, 2019

Seems like there was some improvemen. Here is a plot from Remotery showing time spent in World::step.

step_time_filtered

I added category bits and ODE collision space hierarchies here, which seem to significantly improve
the performance of dSpaceCollide.

@jslee02 jslee02 deleted the grey/use_raw_skeleton_ptr branch March 13, 2024 05:56
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.

4 participants