Add set cardinality etimation

[t0k4rt]

Hello,

I want to implement a set cardinality estimation function
I'm working on a PR (#11) but I cannot manage to launch my unit tests.
Do you have any advice on how to build the project and launch the rust and python test suite ?

I installed maturin, launched maturin develop --release
then tried to launch the unit tests with python -m unittest py_tests.test_bloom.test_bloom_add
but the test fails.

Thanks for your advices

[yankun1992]

Hi, Thanks for your contribution, this project's github action ymal profile may be able to help you.

…

------------------ Original ------------------ From: Alexandre Assouad ***@***.***> Date: Wed,Oct 4,2023 9:41 PM To: yankun1992/fastbloom ***@***.***> Cc: Subscribed ***@***.***> Subject: Re: [yankun1992/fastbloom] Add set cardinality etimation (Issue #12) Hello, I want to implement a set cardinality estimation function I'm working on a PR (#11) but I cannot manage to launch my unit tests. Do you have any advice on how to build the project and launch the rust and python test suite ? I installed maturin, launched maturin develop --release then tried to launch the unit tests with python -m unittest py_tests.test_bloom.test_bloom_add but the test fails. Thanks for your advices — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[t0k4rt]

Hi, thanks for the tip !
The PR is working as expected with the added unit tests

[yankun1992]

Hi, Thanks again for your contribution, but unfortunately I'm on vacation today and I can't deal with this PR until about 2 days later.

…

------------------ Original ------------------ From: Alexandre Assouad ***@***.***> Date: Wed,Oct 4,2023 10:33 PM To: yankun1992/fastbloom ***@***.***> Cc: Yan Kun ***@***.***>, Comment ***@***.***> Subject: Re: [yankun1992/fastbloom] Add set cardinality etimation (Issue #12) Hi, thanks for the tip ! The PR is working as expected with the added unit tests — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: ***@***.***>

[t0k4rt]

No problems, Have a nice vacation !

…

Le 4 oct. 2023 à 17:09, Yan Kun ***@***.***> a écrit : Hi, Thanks again for your contribution, but unfortunately I'm on vacation today and I can't deal with this PR until about 2 days later. ------------------ Original ------------------ From: Alexandre Assouad ***@***.***&gt; Date: Wed,Oct 4,2023 10:33 PM To: yankun1992/fastbloom ***@***.***&gt; Cc: Yan Kun ***@***.***&gt;, Comment ***@***.***&gt; Subject: Re: [yankun1992/fastbloom] Add set cardinality etimation (Issue #12) Hi, thanks for the tip ! The PR is working as expected with the added unit tests — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: ***@***.***&gt; — Reply to this email directly, view it on GitHub <#12 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AADZTOQBDKYY2TVOJDIP6G3X5V33NAVCNFSM6AAAAAA5SU6K7SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBXGA2DSNJYGU>. You are receiving this because you authored the thread.

[yankun1992]

Published 0.5.7