[3d-opengl] Don't take glm objects as parameters or store them

[SeanOMik]

Branch: feature/3d-opengl

A lot of places functions will take in a glm::vec3 (or some other glm type) as a parameter, or a class will store them in an accessible way to the engine user. Other places we're taking in a simpleengine::Vectorf. It would be better to be consistent and use the same type.