#19007: Refactor Closeness Centrality
-------------------------------------+-------------------------------------
       Reporter:  borassi            |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.9
      Component:  graph theory       |   Resolution:
       Keywords:  Closeness          |    Merged in:
  centrality                         |    Reviewers:
        Authors:  Michele Borassi    |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  5c85793ccb8234ac668173620523f45dbae4b985
  u/borassi/refactor_centrality_closeness|     Stopgaps:
   Dependencies:  #18931, #18876,    |
  #18910                             |
-------------------------------------+-------------------------------------

Comment (by dcoudert):

 I don't really understand what you are doing in this commit
 
http://git.sagemath.org/sage.git/commit/?h=5c85793ccb8234ac668173620523f45dbae4b985

 In the ticket description you say that you will move everything in
 `generic_graph`, but it might be more readable/clean to create a new file
 for centrality measures, no?
 Also, you moved only references to the methods, right? (or the
 modifications are hidden in the merge with #18931).
 {{{
 +            - :meth:`~sage.graphs.graph.Graph.centrality_degree`
 +            - :meth:`~centrality_betweenness`
 }}}

 David.

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