Skip to content

Add new AngularVelocity / Acceleration classes that wrap Angle class #3093

@nimazareian

Description

@nimazareian

Description of the task

The AngularVelocity and AngularAcceleration classes are just proxies to the Angle class. This is not great design as not all of the Angle class methods make sense in the case of angular velocity/acceleration. E.g. Angle::clamp does not make sense in the context of AngularVelocity, as 360 deg/s is different from 0 deg/s, but 360 deg is the same as 0 deg.

Acceptance criteria

  • Remove the AngularVelocity and AngularAcceleration classes using the using keyword
  • Create new class that wrap the existing Angle class and only expose the methods that make sense

Blocked By

#3006 adds AngularAcceleration

Metadata

Metadata

Assignees

Labels

Difficulty - 3Easy and simple issues that require some context to do

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions