On Wednesday, May 14, 2014 11:32:10 AM UTC+1, Nathann Cohen wrote:
>
> Yes, but it duplicates the code horribly. Because orthogonal_array is a 
> long sequence of tests and answers
>

Thats another red flag. If it doesn't fit on a screen break it up into 
separate function.
 

> Besides, this is pretty clear :
> sage: designs.orthogonal_array(4,3,existence=True)
> True
>

Its terrible. Misleading function name (doesn't return an array), not 
discoverable. The fact that it also hard to cache is a direct corollary of 
attempting to overload by return type. Don't do it. 

Just because you can (in Python) doesn't mean that its a good idea.

-- 
You received this message because you are subscribed to the Google Groups 
"sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sage-devel+unsubscr...@googlegroups.com.
To post to this group, send email to sage-devel@googlegroups.com.
Visit this group at http://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/d/optout.

Reply via email to