#18811: Boost Clustering Coefficient
-------------------------------------+-------------------------------------
Reporter: borassi | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.8
Component: graph theory | Resolution:
Keywords: Local clustering | Merged in:
coefficient, Boost | Reviewers: Nathann Cohen
Authors: Michele Borassi | Work issues:
Report Upstream: N/A | Commit:
Branch: | 6c8d49bc6d0101f897c489124d52c5e4a252a959
u/borassi/boost_clustering_coefficient| Stopgaps:
Dependencies: 18564 |
-------------------------------------+-------------------------------------
Comment (by borassi):
Sorry, I didn't realize you modified the code... That's why I modified the
doc, I thought you forgot a commit, or something like that. In my opinion,
it is better to modify the doc, for simplicity!
BTW, how can I use 'tolerance' in the other test?
{{{
sage: G = graphs.RandomGNM(10,20)
sage: clust_boost = G.clustering_coeff(implementation='boost')
sage: clust_networkx = G.clustering_coeff(implementation='networkx')
sage: for v in G.vertices():
....: if abs(clust_boost[v] - clust_networkx[v]) > 1E-12:
....: print "Error:"
....: print " Boost clustering of", v, "is", clust_boost[v]
....: print "NetworkX clustering of", v, "is", clust_networkx[v]
}}}
Replying to [comment:17 ncohen]:
> Hey nononon! I had changed the code to handle the 'None' case! Do
whatever you want, but please keep the code and the doc consistent.
>
> Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/18811#comment:18>
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.
