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

Add bloom filter #84

Merged
merged 2 commits into from
Jun 30, 2018
Merged

Conversation

arnav-aggarwal
Copy link
Contributor

No description provided.

@arnav-aggarwal
Copy link
Contributor Author

I've written my own implementation of a bloom filter in JavaScript. Let me know if you'd like any changes. I'm happy to help.

@codecov-io
Copy link

codecov-io commented Jun 27, 2018

Codecov Report

Merging #84 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #84   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          98     99    +1     
  Lines        1984   2020   +36     
  Branches      352    353    +1     
=====================================
+ Hits         1984   2020   +36
Impacted Files Coverage Δ
src/data-structures/bloom-filter/BloomFilter.js 100% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update db7ab9e...58da35e. Read the comment docs.

@trekhleb trekhleb changed the base branch from master to bloom-filter June 30, 2018 17:06
@trekhleb
Copy link
Owner

@arnav-aggarwal great work! Thank you!

@trekhleb trekhleb merged commit 41a6430 into trekhleb:bloom-filter Jun 30, 2018
shoredata pushed a commit to shoredata/javascript-algorithms that referenced this pull request Mar 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants