Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FLINK-2375] Add Approximate Adamic Adar Similarity Measure Example t… #923

Closed
wants to merge 1 commit into from

Conversation

shghatge
Copy link
Contributor

…o Gelly

Implementation of Adamic Adar Similarity Measure using BloomFilters

@tillrohrmann
Copy link
Contributor

I think you've referenced the wrong jira ticket.

@shghatge shghatge changed the title [FLINK-2075] Add Approximate Adamic Adar Similarity Measure Example t… [FLINK-2375] Add Approximate Adamic Adar Similarity Measure Example t… Jul 20, 2015
@andralungu
Copy link
Contributor

Could you also do a git ammend to reference the correct jira issue in the commit?

@shghatge
Copy link
Contributor Author

Updated PR
@vasia I removed the Vertex Centric Iteration :) Hope this method is okay.

@vasia
Copy link
Contributor

vasia commented Jul 21, 2015

Hi @shghatge!

The comments I left on #892 apply here as well. The difference would be that the neighborhoods of each of the neighbors will be represented as a bloom filter.

It would also be nice to make the bloom filter parameters (size, number of hash functions, hash function) configurable, so that the user can adjust the false positives and size based on their use-case.
What do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants