pik-copan / pyunicorn

Unified Complex Network and Recurrence Analysis Toolbox
http://pik-potsdam.de/~donges/pyunicorn/
Other
195 stars 87 forks source link

Initial commit to include directed degree measures in the network class #63

Closed wbarfuss closed 8 years ago

wbarfuss commented 9 years ago

New methods =nsi_indegree= and =nsi_outdegree= introduced so far. Open Questions / Points to discuss

wbarfuss commented 9 years ago

Hi, two things: 1) Unfortunately it turned out that it is much more suitable to work on the issue regarding Delphines paper in only one branch. So I started by now implementing the clustering motifs here as well, although the branch is called nsi_degree. I don't know if we should or can rename something? 2) pylint discovers two new "error": C: 1, 0: Too many lines in module (5670/5600) (too-many-lines) R:122, 0: Too many public methods (113/110) (too-many-public-methods) How shall we proceed with this?

wbarfuss commented 9 years ago

Another point: local_clustering works with directed and undirected networks by treating the directed as undirected links. nsi_local_clustering works only with undirected networks. Shall this be harmonized?

ntfrgl commented 9 years ago
wbarfuss commented 9 years ago

I think, that would solve than issue #49

ntfrgl commented 9 years ago

Please use tests.test_doctests.r() in doctests with floats, see e.g. Network.local_clustering().

wbarfuss commented 9 years ago

I hope I didn't forget any