-
-
Notifications
You must be signed in to change notification settings - Fork 113
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
square norm in pointvector #1365
Conversation
oops.. this PR also contains @JacquesOlivierLachaud's commits on #1364 |
src/DGtal/kernel/PointVector.h
Outdated
* | ||
* @return the square norm of the point/vector as a double. | ||
*/ | ||
double sqnorm( ) const; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason for choosing the name "sqnorm" instead of "squaredNorm" or "squaredL2Norm" ? Because it is not our usual naming conventions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i've seen this name in previous lib.. But I can switch to squaredNorm is you want
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
squaredNorm is more homogeneous with DGtal code, but has the disadvantage to be longer (sometimes a problem for reading the code)... I let you decide.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's go with squaredNorm
All good. Merging. |
PR Description
Checklist
cmake
mode (otherwise, Travis C.I. will fail).