Skip to content
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

Use separate bolt density #5

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nepfaff
Copy link
Contributor

@nepfaff nepfaff commented Jan 7, 2025

This doesn't really make a difference, but it seems like the right thing to do. This is based on the linked bolts in the README.

Before:

python3 GenerateObjectConfiguration.py -b 0 14 -s 0 1 2 3 4 -o test.urdf --print
test.urdf
Total Mass:      1.1718997410246232
Center of Mass:  [0.0000000000 0.0409352866 0.0250000000]
Inertia Tensor wrt. origin, about COM:
 [[0.0171855735 0.0000000000 0.0000000000]
 [0.0000000000 0.0040224473 -0.0023986026]
 [0.0000000000 -0.0023986026 0.0179134108]]

After:

python3 GenerateObjectConfiguration.py -b 0 14 -s 0 1 2 3 4 -o test.urdf --print
test.urdf
Total Mass:      1.1680782668800922
Center of Mass:  [0.0000000000 0.0410692101 0.0250000000]
Inertia Tensor wrt. origin, about COM:
 [[0.0171004315 0.0000000000 0.0000000000]
 [0.0000000000 0.0040168639 -0.0023986026]
 [0.0000000000 -0.0023986026 0.0178338314]]

@nepfaff nepfaff marked this pull request as ready for review January 7, 2025 17:31
@PhilNad PhilNad mentioned this pull request Jan 8, 2025
@PhilNad
Copy link
Contributor

PhilNad commented Jan 8, 2025

Indeed, since the bolt is modeled as a cylinder but is not a perfect cylinder, its mass will be slighter less than was would be obtained from the steel mass density and the cylindrical shape.

Also, currently, the head and nut of the bolt are not taken into account (as you can see their impact is very small). I wonder what would be the best approach to model the bolts without spending too much effort on very small details.

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.

2 participants