dilevin / CSC417-a5-rigid-bodies

14 stars 5 forks source link

Slight discrepancy in center of mass definition #3

Open abhimadan opened 4 years ago

abhimadan commented 4 years ago

According to Wikipedia, the center of mass of a volume is the [X, Y, Z] integral we see divided by the total mass. This means that [X, Y, Z] is actually not the center of mass in general (at least based on how those starred quantities are defined in the assignment), but rather a scalar multiple of the center of mass. It happens to make no difference computationally in the assignment because the (constant) density is 1, but this is still a bit surprising. Can the readme be changed to reflect this?

dilevin commented 4 years ago

Fixed, I forgot a 1/m.