#16227: Product construction of Transversal Designs
-------------------------+-------------------------------------------------
       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:         |  a512ab154b21caaf74d6ccef2cf32bf67179eee9
  Nathann Cohen          |     Stopgaps:
Report Upstream:  N/A    |
         Branch:         |
  u/ncohen/16227         |
   Dependencies:         |
  #15310                 |
-------------------------+-------------------------------------------------

Comment (by ncohen):

 > Sometimes you want only functions with the same prototype and sometimes
 you claim it is better otherwise...

 1) I think that having a TD product functions that accepts TD as input and
 not only parameters is useful

 2) I think that at the moment Wilson's construction with TD as input is
 not really useful, as it is a complex thing and nobody knows it exists

 3) I think that all constructors should have the same prototype to give us
 a clean `orthogonal_array` function

 4) Thus creating a second function that calls the current TD and has the
 same prototype as the others makes sense to me.

 > All right. What I meant is that it makes sense to try Wilson and product
 in the same function that builds OA. But on the other hands, calling
 divisors is relatively cheap on the kind of entries you will have. In a
 near future (and according to the TODO) the Wilson construction might be
 much more involved... will see.

 Calling divisors is 100% free compared to the kind of constructions we
 deal with. We create fields, products, long lists, temporary matrices of
 thousands of entries. Calling divisors *is* free.

 > There was doctest error that I solved on u/vdelecroix/16227. I also
 added curiosity doctests (the current maximum k that Sage knows from n
 between 10 and 20).
 >
 > See the changes in u/vdelecroix/16227, if you are happy with them this
 is good to go.

 I will look at this immediately.

 Nathann

--
Ticket URL: <http://trac.sagemath.org/ticket/16227#comment:9>
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