#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       |  ba058dd515c38084b4cdeacf530eda7b4563d211
   Dependencies:  #15310, #16227     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by ncohen):

 Yo !

 >  - implement my own `projective_space_as_OA` inside
 `orthogonal_arrays.py`. This might sound weird but the current
 implementation of `ProjectiveSpaceDesign` is slow and scarry me (I plan to
 clean it soon in a ticket).

 THis thing has *NOTHING* to do here. Use the current
 `ProjectivePlaneDesigns`, and write another ticket to change the code if
 you don't like it. I agree that it is slow, but implementing the function
 TWICE (one that is slow, one that is hidden in an unrelated module) is not
 a way to solve it.

 >  - implement a beautiful `TD_existence` (I do not like the name, so I
 aks for suggestions here):

 Beautiful ? Is that a joke ? A function that PRINTS text ?

 Get this thing out of here until a proper interface design is found. A
 function like that should RETURN values, not print stuff. Plus we have
 almost no negative result to advertise for the moment. Plus how the hell
 is an user going to find that ? We write Sage code here, not our own
 scripts !!!

 If such a function is to ever be implemented, it will return a pair "lower
 bound, upper bound". The only interesting data in your three lines are TWO
 NUMBERS, and wrapping them in intervals for nothing, adding a 2 there and
 an infinity there looks fancy, but once more we work on a library here,
 not on your own code.

 > I also correct some of the errors raised to get mainly `EmptySetError`
 and `NotImplementedError`.


 I will look at that when the problems above are solved....

 Nathann

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