First, i've generalized the angle data type of existing magnetic encoder driver as5047.
👉 A consequent follow up would be to integrate driver/encoder/angle.hpp with math/geometry/angle.hpp. F.e. by adding conversion constructors to math/geometry/angle.hpp and drop the redundant methods ::toRadian() and ::toDegree() in driver/encoder/angle.hpp. But we might also just keep it as it is.
Driver for a small and cheap magnetic encoder.
First, i've generalized the angle data type of existing magnetic encoder driver as5047. 👉 A consequent follow up would be to integrate
driver/encoder/angle.hpp
withmath/geometry/angle.hpp
. F.e. by adding conversion constructors tomath/geometry/angle.hpp
and drop the redundant methods ::toRadian() and ::toDegree() indriver/encoder/angle.hpp
. But we might also just keep it as it is.What do you think?