-
Notifications
You must be signed in to change notification settings - Fork 200
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
move from ofVec3f to glm #300
base: master
Are you sure you want to change the base?
Conversation
@OmerShapira it would be nice if you can have a look ;) |
Also, the format of the formula it is broken since a while now. @OmerShapira Which tool did you use to format it? is it possible for you to fix it? otherwise I think it make sense to remove it. |
ping @OmerShapira |
I think it makes sense to remove the broken formulas. @arturoc ? |
the syntax highlighting error is related to #291. |
the rendering issue is server side I'd wager... I seem to remember that certain packages are needed. I would guess running the script that builds the book would give some insight |
@edap I think what we could do is leave this document as it is and add your modifications as a new one + a note at the beginning of this one explaining that since 0.10 OF uses glm instead of our own implementation of vector math. There's still a lot of projects and addons that use ofVec so i think it would be a good idea to keep both versions around. |
Main changes:
Regarding:
I have removed it because in glm there is not a function out of the box to do this. Although we could implement it, look at http://www.opengl-tutorial.org/intermediate-tutorials/tutorial-17-quaternions/ and “RotationBetweenVectors”. In case it is something that we would like to have, i can make a PR for adding this function, that it is pretty useful (I have tested the exact snipper in my own projects).