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

added interface to set fpRate #28

Merged
merged 1 commit into from
Mar 24, 2023

Conversation

kcw-grunt
Copy link
Member

Overview

Allows the user to chose a fp for the bloom filter

Closes
#27

fpRate: false positive rate for the bloomfilter
@kcw-grunt
Copy link
Member Author

ugh...no looks. I need to merge. Its been 3 days.

@kcw-grunt kcw-grunt merged commit d09cce2 into master-android Mar 24, 2023
@kcw-grunt kcw-grunt deleted the add-false-positive-func-27 branch March 24, 2023 07:34
@josikie
Copy link
Member

josikie commented Mar 24, 2023

@kcw-grunt Sorry, I just opened it. You are doing great. Btw, I have questions, what are BRSetApply, BRSetClear, BRAddress?

kcw-grunt added a commit that referenced this pull request Mar 29, 2023
* typo :)

* va_list include

* enable ipv6

* default to ipv6 and fallback to ipv4

* fix ipv4 fallback

* Increase MAX_FEE_PER_KB

* Fix 0 change output

* sha3-256 and keccak-256

* const keywords

* update tx ordering on confirmation

* compiler warnings

* Litecoin: params

* Litecoin: scrypt pow & max_proof_of_work

* Litecoin: disable retarget check

* Litecoin: add new checkpoint

* BIP39_CREATION_TIME -> LoafWallet release date

* update secp

* make sure to call balanceChanged callback whenever balance is updated

* Change bitcoin-testnet with litecoin-testnet

* bech32: update files

* add missing files

* add BRPeerManagerStandardPort method which returns the standard port used for chain params

* 🦺Updated the checkpoints: Android branch (#24)

* Updated the checkpoints

* only 1 checkpoint added

* Added Testnet checkpoint

* added interface to set fpRate (#28)

fpRate: false positive rate for the bloomfilter

* fixed the signature (#29)

---------

Co-authored-by: LiteBit <[email protected]>
Co-authored-by: Aaron Voisine <[email protected]>
Co-authored-by: Adrian Corscadden <[email protected]>
Co-authored-by: Luca Vaccaro <[email protected]>
Co-authored-by: Loshan T <[email protected]>
Co-authored-by: Mohamed Barry <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants