#19545: Mathon's pseudocylic strongly regular graphs.
-------------------------+-------------------------------------------------
       Reporter:         |        Owner:
  dimpase                |       Status:  new
           Type:         |    Milestone:  sage-6.10
  enhancement            |   Resolution:
       Priority:  major  |    Merged in:
      Component:  graph  |    Reviewers:
  theory                 |  Work issues:
       Keywords:         |       Commit:
        Authors:  Dima   |  af1af3242be804d31bf81ecc0db94f7378b0f152
  Pasechnik              |     Stopgaps:
Report Upstream:  N/A    |
         Branch:         |
  u/dimpase/matpc        |
   Dependencies:         |
-------------------------+-------------------------------------------------

Comment (by ncohen):

 A couple of remarks:

 - Bad indentation in the 'reference' blocks.
 - This is ridiculous
   {{{
   sum(map(lambda i: P[a.index(g**(2*i))], xrange(1, 2*t)))
   }}}
   Forget these lambda functions and use Python. It will be easier to read,
 and less costly in memory/computations. What you want is
 `sum(P[a.index(g**(2*i))] for i in range(1,2*t))`.

 - You did the right thing when you defined the function `B(m)`. Please do
 the same for the lambda function that you use to define `A`.

 Nathann

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