On Sep 22, 2012, at 11:12 AM, Volker Braun wrote:

> On Saturday, September 22, 2012 2:11:08 AM UTC+1, Travis Scrimshaw wrote:
> normal_form() is good, and to be standard with the other code packages I've 
> seen, dividing by the L2-norm should be normalize().
> 
> As Robert pointed out, normalize() is bad because the name implies that the 
> vector is mutated (which is not what the method does). It should have been 
> normalization() from the start.
> 
> So the updated proposal is:
> 1) deprecate normalize() and point to normalization() and normal_form()
> 2) normalization() is division by L2-norm
> 3) normal_form() is the old normalize()
> 4) alias norm() -> normalization() to save typing

Isn't v.norm() supposed to denote the… norm of v ? (the docstring says that by 
default it is the L2 norm). I would find it quite confusing that v.norm() 
returns a vector instead of the norm itself (a scalar)

Charles

-- 
You received this message because you are subscribed to the Google Groups 
"sage-devel" group.
To post to this group, send email to sage-devel@googlegroups.com.
To unsubscribe from this group, send email to 
sage-devel+unsubscr...@googlegroups.com.
Visit this group at http://groups.google.com/group/sage-devel?hl=en.


Reply via email to