#13075: Toroidal6RegularGrid2dGraph
---------------------------------+------------------------------------------
       Reporter:  ncohen         |         Owner:  jason, ncohen, rlm
           Type:  enhancement    |        Status:  needs_work        
       Priority:  major          |     Milestone:  sage-5.4          
      Component:  graph theory   |    Resolution:                    
       Keywords:                 |   Work issues:                    
Report Upstream:  N/A            |     Reviewers:  David Coudert     
        Authors:  Nathann Cohen  |     Merged in:                    
   Dependencies:  13067          |      Stopgaps:                    
---------------------------------+------------------------------------------
Changes (by {'newvalue': u'Nathann Cohen', 'oldvalue': ''}):

  * status:  needs_review => needs_work
  * reviewer:  => David Coudert
  * author:  => Nathann Cohen


Comment:

 Hello,

 I was about to write a small review patch to add:
 - a space between i and \in (same for j) since otherwise i and in are
 glued when asking for the doc of the function.
 - to write {{{`((i+1)%n_1, (j+1)%n_2`}}} so add ().
 But then I found a problem in the method as well as for
 {{{ToroidalGrid2dGraph}}}, {{{Grid2dGraph}}}, and certainly others: I'm
 able to call {{{graphs.Toroidal6RegularGrid2dGraph(-1,2)}}} without
 warnings! O_o

 For which values of n1 and n2 are these methods defined? So far, I'm able
 to produce a graph that is not 6-regular.

 David.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13075#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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to