#15431: Transversal Design TD(6,12)
-------------------------+-------------------------------------------------
       Reporter:         |        Owner:
  ncohen                 |       Status:  needs_review
           Type:         |    Milestone:  sage-6.2
  enhancement            |   Resolution:
       Priority:  major  |    Merged in:
      Component:         |    Reviewers:
  combinatorics          |  Work issues:
       Keywords:         |       Commit:
        Authors:         |  c87cac5f35036a820f877ab891eb35cffa1784a1
  Nathann Cohen          |     Stopgaps:
Report Upstream:  N/A    |
         Branch:         |
  u/ncohen/15431         |
   Dependencies:         |
  #15287 #15368          |
-------------------------+-------------------------------------------------

Comment (by vdelecroix):

 Hi Nathan,

 Few remarks that would me help me for the review

  * In the function  transversal_desing
    * the arguments are ordered k,n but in the definition n,k is used.
    * could you give the definition with t

  * In the function orthogonal_arrays
    * you should defined what is a (k,n,t) orthogonal array.
    * the documentation says that only t=2 is available but it does not
      prevent you to define it in that generality.
    * I guess that if k >= n+t there is no (k,n,t) orthogonal array. Do
      you know that ? If this is true, please more explicit in the error
      rather than `NotImplementedError`: I don't know how to build this
      orthogonal array!" I would prefer "`EmptySetError`: there exist no
      (k,n,t) orthogonal array with k >= n+t"

--
Ticket URL: <http://trac.sagemath.org/ticket/15431#comment:5>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

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

Reply via email to