flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From vasia <...@git.apache.org>
Subject [GitHub] flink pull request: [FLINK-1528][Gelly] Added Local Clustering Coe...
Date Sat, 07 Mar 2015 12:37:40 GMT
Github user vasia commented on the pull request:

    https://github.com/apache/flink/pull/420#issuecomment-77687686
  
    Hi @balidani! Thanks for the example :))
    I'm a bit confused about the directed / undirected case. I tried testing with both a directed
and an undirected input and both my tests failed.. See [here](http://gist.github.com/vasia/ad68268118b6f895cf7c#file-lcc-example-tests)
are the test cases I tried.
    I suppose the directed case doesn't work because you only consider the out-neighbors,
when you should count all. And for the undirected case, I think that the division by 2 you're
making is giving a wrong result, because you're not counting edges, just out-neighbors neighbors.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message