Rename Velocity
properties linvel
and angvel
to linear
and angular
#505
Labels
Velocity
properties linvel
and angvel
to linear
and angular
#505
Velocity
currently has thelinvel
andangvel
properties. This often leads to repetition of "vel". For example, one of the examples in the user guide:I find this repetition rather annoying and unnecessary given that the component is already called
Velocity
. Abbreviating "linear velocity" to "linvel" also doesn't seem useful, and might be confusing to new users. In general, abbreviated type names are not recommended.I propose renaming the properties to simply
linear
andangular
, increasing clarity and readability.The earlier example would look like this:
(I would also change
vel
tovelocity
, especially in documentation, but that is unrelated to the proposal)Alternatively, we could split
Velocity
toLinearVelocity
andAngularVelocity
likebevy_xpbd
, but that would be a larger breaking change and have the caveat of methods likelinear_velocity_at_point
not really working since they need access to both at once.The text was updated successfully, but these errors were encountered: