Updated the documentation to mention the HomogRotate3D requires a normalised axis vector.

go-gl / mathgl

A pure Go 3D math library.

BSD 3-Clause "New" or "Revised" License

554 stars 65 forks source link

Updated the documentation to mention the HomogRotate3D requires a normalised axis vector. #69

Closed bramp closed 6 years ago

bramp commented 6 years ago

OpenGL's glm does not require a normalised vector, and porting some C++ to Go bite me. I've also added a couple of tests for this behaviour, which I confirmed was correct by comparing to WolframAlpha

UserAB1236872 commented 6 years ago

Ah, I wasn't aware that GLM didn't require normalization. I wanted to avoid it for performance. This is a good change, thanks.