Optic flow / code redefinition
Hi,
I realised that you are using OFSubroutineSPToVector
. This function is not necessary, because in maths/coordinates we have function to convert between spherical and cartesian coordinates.
For Matrix_transform
, it seems misleading to do a rotation of opticFlow within this function. It should be placed in opticflow function. Moreover, I would rename Matrix_transform into a more meaningful name e.g. cartesian_to_spherical_vectors. This function could be move to the maths/convertions because it is not specific to optic flow, but apply to any vectors