#16403: Remove Graph.to_partition
-------------------------+-------------------------------------------------
       Reporter:         |        Owner:  pdehaye
  ncohen                 |       Status:  needs_review
           Type:         |    Milestone:  sage-6.3
  defect                 |   Resolution:
       Priority:  major  |    Merged in:
      Component:  graph  |    Reviewers:
  theory                 |  Work issues:
       Keywords:         |       Commit:
        Authors:         |  9f08c6b859a0fb853f69dbaedc73811740ee3e91
  Nathann Cohen          |     Stopgaps:
Report Upstream:  N/A    |
         Branch:         |
  u/ncohen/16403         |
   Dependencies:         |
-------------------------+-------------------------------------------------

Comment (by ncohen):

 > Not done. Your first sentence still shows indecision on exactly what to
 do and why to do it.

 Are we really discussing my writing style here ?

 > Why is the burden on me to explain why it is useful?

 Because if it is not we will remove it, like any old code we don't want to
 carry anymore.

 > The burden should be on you to first explain that this is harmful.

 I do not consider this harmful, I consider this useless. We also have
 reasons to believe that it has been added without wondering whether it
 would be useful, which gives me more reasons to think that it is useless.

 > Judging from the mailing list and what I know, I think this is useful.

 Can't do anything against pure faith. Do you only accept claims without
 proofs from yourself or do you intend to be as explicit as you expect me
 to be ? This code can be written in one line, there is no doubt on earth
 that the only reason to implement this function was its decorator.
 Besides, with a name like that nobody even knows it exists and what it
 does.

 It does not even appear in the index of graph functions :
 http://www.sagemath.org/doc/reference/graphs/sage/graphs/graph.html

 > Be careful when publicly doubting someone's intellectual honesty. There
 is a fine line there. You might want to take a breather.

 Man, we are discussing how bits should be arranged in a part of Sage's
 code that 5 people on earth know to exist. Stop taking this seriously.

 Nathann

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