#18972: twographs and Seidel switching
-------------------------------------+-------------------------------------
       Reporter:  dimpase            |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.9
      Component:  graph theory       |   Resolution:
       Keywords:                     |    Merged in:
        Authors:                     |    Reviewers:  Nathann Cohen
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/dimpase/seidelsw                 |  25eec1b6d18bb021eccd4b04774111097ff8f3b5
   Dependencies:  #18960, #18948,    |     Stopgaps:
  #18988, #18991, #18986, #19018,    |
  #19019                             |
-------------------------------------+-------------------------------------

Comment (by ncohen):

 > well, `designs.TAB` gives
 > {{{
 > <lots of things>
 > }}}
 >
 > Quite a few of them added by you, right?

 Yes. I am quite proud of that.

 > Why do you ask me to make `TwoGraph` a second-class citizen then?

 No. I asked you to *not make it appear in the catalog*, in the same way
 that you do not see the following classes (that I also created):
 `BalancedIncompleteBlockDesign`, `GroupDivisibleDesign`,
 `PairwiseBalancedDesign`.

 Catalogs are not meant to hold generic class constructors (you don't see
 Graph in 'graphs.' or 'PermutationGroup' in `group.`, but only functions
 meant to return a "famous object". That's what catalogs are for. And
 'TwoGraph', like 'BalancedIncompleteBlockDesign', is to be instanciated
 with a list of blocks (like `IncidenceStructure` or
 `BalancedIncompleteBlockDesign` while orthogonal arrays (which appear in
 `designs.<tab>` are created from a pair of integers.

 I mean. That's what catalogs are supposed to hold. You have `matrix.ones`
 gut not `matrix.Matrix`.

 Nathann

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