#16272: redesign transversal designs
-------------------------------------+-------------------------------------
       Reporter:  vdelecroix         |        Owner:  Vincent Delecroix
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.2
      Component:  combinatorics      |   Resolution:
       Keywords:  designs,           |    Merged in:
  orthogona arrays                   |    Reviewers:
        Authors:  Vincent Delecroix  |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:  public/16272       |  d81f265637099f93495dafac54e2d354be0d66d5
   Dependencies:  #15310, #16227,    |     Stopgaps:
  #16281                             |
-------------------------------------+-------------------------------------

Comment (by ncohen):

 > I do not care too much about calling twice the function.

 Cool. Then I changed the code  to have a nice "if/elif" structure. In some
 very specific cases the projective plane function is called three times,
 but

 1) This is only when the answer has been found, so it does not happens
 often per call

 2) A tricky replacement is possible too, as
 {{{
     elif (t == 2 and
           (projective_plane(n, existence=True) or
            (k == n+1 and projective_plane(n, existence=True) is False))):
 }}}

 is equivalent to:

 {{{
     elif (t == 2 and
           (projective_plane(n, existence=True) in (True,k<n+1))):
 }}}

 This being said the code of these things compared to the cost of building
 the actual array ...

 > We leave the idea of having an optional `k=None`? Might be an
 independent ticket anyway.

 Yep, let's close this one and get #16277 in. I will merge it in a second.

 Nathann

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