#16362: Orthogonal Polar Graph
-------------------------------------+-------------------------------------
       Reporter:  ncohen             |        Owner:
           Type:  enhancement        |       Status:  positive_review
       Priority:  major              |    Milestone:  sage-6.3
      Component:  graph theory       |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Nathann Cohen      |    Reviewers:  Dima Pasechnik
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  public/ticket/16362                |  ff7d3387b4436411ed6785e2a3b6c5c37f2ca1f6
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by dimpase):

 Replying to [comment:43 ncohen]:
 > > I don't understand your position. If you want to sneak your worse code
 through the back door, I will protest.
 >
 > It is very simple : I work with this code, I need it to be fast. If a
 small change makes it go faster, I do it.

 The code is as fast in my version as it is in yours. Your measurements are
 not correctly done.
 Do I need to explain why, again? Or do I need to raise it up on sage-
 develop? We can do the latter, sure.

  The disadvantage of your code is that it is entirely possible that the
 pexpect interface gets (half)broken yet again.
 Do I need to explain this? Or you just won't listen, because you think you
 know better?
 As well, let me point out that I find your suggestion that you will get
 rid of my changes in some way that I won't notice very, very worrying.
 This is not the way things are meant to be done.

 And, finally, the design of the these tickets is very suboptimal, anyway,
 as they should be done with a backend that can take advantage of rich
 symmetries of the objects. I think I explained this on more than one
 occasion.
 GAP's GRAPE package can easily deal with graphs from this family with 10
 times as many vertices as your code here.
 That's why I find your worry about a couple of more seconds on startup
 just funny...

 >
 > You would be surprised to see how many people also think that it makes
 sense.

 You might be surprised to find how many people find your outburst like
 this one very immature.

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