-
Notifications
You must be signed in to change notification settings - Fork 2
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
Setup profiler #52
Setup profiler #52
Conversation
Signed-off-by: Louise Poubel <[email protected]>
This is great, glad to see such a tool exists. I was surprised that the ElectroHydraulicPTO plugin didn't win the award. I know for sure there's some optimizations in there still to chase. |
Most likely my debug console printouts. I can remove most of them now. |
@chapulina when I try this, I don't see SimulationRunner in the Remotery window |
Did you compile all of Ignition Fortress from source with Other tips to navigate the UI may also help here. Use the scroll wheel to zoom in a bit more time-wise, and click on the |
Ah, that's the issue. Thanks! |
Signed-off-by: Louise Poubel <[email protected]>
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.
👍
It also might be worth mentioning that I had to run:
colcon build --cmake-args ' -DBUILD_TESTING=true' ' -DENABLE_PROFILER=0'
in my workspace to get colcon test
to work again (even on other branches... since it's in the cmake in build
folder in top level workspace)
Ahh good call, it took me a while to figure that out the first time I needed it, it's definitely worth mentioning. |
Ignition Common's profiler is very handy to debug performance.
This PR just sets up the profiler and makes use of the existing profiling points.
See the snippet added to the README for how to enable and use it. In order to get profiling points coming from the Gazebo libraries themselves, they need to be compiled from source with
-DENABLE_PROFILER=1
.Here's some example output. I was surprised to see that the
PolytropicPneumaticSpring
is taking up almost as much time as the physics' forward step.