-
Notifications
You must be signed in to change notification settings - Fork 91
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
HMM_LengthSquareRoot is poorly named #33
Comments
Oh! That should be changed then! |
Changing the name has the potential to break projects though. Are we ok with that since we're still in early release, or would we want to mark it as deprecated somehow...? (I vote for just breaking things.) |
Were pre 1.0 were fine to break things as long as put a notice in that we broke something |
StrangeZak
pushed a commit
that referenced
this issue
Aug 30, 2016
Fixed in #37. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This is not a bug, exactly, but when I was integrating Handmade Math into my project I was confused by the naming of
HMM_LengthSquareRoot
. I expectedHMM_LengthSquared
, or something, since the function actually produces the length of the vector squared. (No square root has happened yet!)The text was updated successfully, but these errors were encountered: