osrf / drcsim

Repository for the DRC project.
16 stars 6 forks source link

Building VRCPlugin.so from source results in much larger .so #463

Closed osrf-migration closed 9 years ago

osrf-migration commented 9 years ago

Original report (archived issue) by GoRobotGo (Bitbucket: GoRobotGo).


In building drcsim from source I noticed that VRCPlugin.so I built was much larger than the original one released with the Jan 3rd pre-release. It also resulted in gazebo running about 10% slower. I assume the same may be true of other libraries. Original: -rw-r--r-- 1 root root 468328 Jan 4 09:33 libVRCPlugin.so Newly compiled: -rw-r--r-- 1 root root 4097817 Jan 5 09:58 libVRCPlugin.so

Followed instructions on building from source, but switched to 4.2 prerelease using: hg checkout drcsim_4.2_prerelease

Also tried -DCMAKE_BUILD_TYPE=MinSizeRel and -DCMAKE_BUILD_TYPE=Release

Any ideas what I am doing wrong to make my libVRCPlugin.so so much bigger?

osrf-migration commented 9 years ago

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


I see the same thing. I'm guessing that symbols are stripped out of the library during the debian build process?

@_jrivero_

osrf-migration commented 9 years ago

Original comment by Ian Chen (Bitbucket: Ian Chen, GitHub: iche033).


I believe the 10% performance hit is from the increase in ode solver iterations from 50 to 100. This is done to improve atlas v5 walking stability. Actually no, if you're already on 4.2.3 (source or deb) then you have the same physics settings.

osrf-migration commented 9 years ago

Original comment by John Hsu (Bitbucket: hsu, GitHub: hsu).


It's done automatically by cmake, the symbols gets stripped out during the install process. If you did catkin_make install, you can compare libVRCPlugin.so in your /build/ directory vs the one installed in /install/ and see the different sizes. This should not affect operation.

Maybe @j-rivero has some more insights, but I'll close this issue for now.

osrf-migration commented 9 years ago

Original comment by John Hsu (Bitbucket: hsu, GitHub: hsu).