joinmarket icon indicating copy to clipboard operation
joinmarket copied to clipboard

configurable thresholds for utxo merge policies

Open adlai opened this issue 9 years ago • 16 comments

see #91 and #173, specifically this comment

utACK :sweat_smile:

adlai avatar Feb 02 '16 13:02 adlai

rebased + removed unicode

adlai avatar Feb 04 '16 09:02 adlai

Untested but I agree with it.

Aside: The comments in the default joinmarket.cfg file seem to be getting quite large and wordy, I wonder if it's worth documenting the options on the wiki instead, with only a link in .cfg

Bikeshed: the default threshold should be higher for privacy IMO. Use bitcoin.select() up to about 30 UTXOs, switch to gradual around 40, greedy at 50 and greediest at 60+. My yield generator wallet has about 65 UTXOs if anyone is wondering. Having lots of UTXOs is great for privacy, its like owning a total of 20,000 CHF in a mixture of banknote sizes from 5 CHF all the way up to 1000 CHF notes. Spending them leaks a lower bound of your balance.

chris-belcher avatar Feb 09 '16 12:02 chris-belcher

Re: bikeshed, bumped the numbers up and amended the comment.

adlai avatar Feb 10 '16 10:02 adlai

Has anyone tried this yet? It could be used on someone's yield generator.

chris-belcher avatar Mar 27 '16 00:03 chris-belcher

I'd say lets merge this into develop and just fix it if it crashes when being used, develop is allowed to be slightly unstable.

chris-belcher avatar Apr 10 '16 15:04 chris-belcher

I'd say lets merge this into develop and just fix it if it crashes when being used, develop is allowed to be slightly unstable.

I sort of agree, but I just wrote #496 and I think it applies here. If we want to make a new release, is it needed to add this?

If it is, please provide test coverage of some sort.

AdamISZ avatar Apr 10 '16 16:04 AdamISZ

Yes fair enough, write a test

This is still worth merging as a default because JoinMarket takers dont really have a way to handle very large amounts of UTXOs yet so this would be good to help prevent them happening in the first place.

chris-belcher avatar Apr 10 '16 16:04 chris-belcher

rebased onto current develop. travis failure is mysteriously inscrutable, to me at least.

test pending.

adlai avatar Jan 03 '17 02:01 adlai

That test failure is confusing to me as well. Got no idea.

chris-belcher avatar Jan 10 '17 18:01 chris-belcher

@adlai Looks like a circular dependency because you imported bitcoin into support.py

AdamISZ avatar Jan 11 '17 21:01 AdamISZ

Coverage Status

Coverage increased (+0.2%) to 77.473% when pulling 76567cd86590547e16c77d2a44d1af3d43b3b53a on adlai:merge-policies into 8c2b6d83dfddb85451d185e64f0fe4df62d5c54e on JoinMarket-Org:develop.

coveralls avatar Apr 17 '17 05:04 coveralls

Coverage Status

Coverage increased (+0.2%) to 77.503% when pulling 3b3bca9f45cbde3113044810d938d86ab50404f4 on adlai:merge-policies into 8c2b6d83dfddb85451d185e64f0fe4df62d5c54e on JoinMarket-Org:develop.

coveralls avatar Apr 17 '17 06:04 coveralls

Coverage Status

Coverage increased (+0.2%) to 77.488% when pulling 3b3bca9f45cbde3113044810d938d86ab50404f4 on adlai:merge-policies into 8c2b6d83dfddb85451d185e64f0fe4df62d5c54e on JoinMarket-Org:develop.

coveralls avatar Apr 17 '17 06:04 coveralls

Coverage Status

Coverage increased (+0.4%) to 78.552% when pulling 099973b141d652bf0b2fa9b7970007959d021531 on adlai:merge-policies into 46fe3e8bbe9bbc769db1732029e90db9591fe8d5 on JoinMarket-Org:develop.

coveralls avatar Oct 03 '17 08:10 coveralls

Coverage Status

Coverage increased (+0.4%) to 78.49% when pulling 729df6c92ff06336199437fced8ae83ab1c96263 on adlai:merge-policies into 46fe3e8bbe9bbc769db1732029e90db9591fe8d5 on JoinMarket-Org:develop.

coveralls avatar Oct 03 '17 09:10 coveralls

Coverage Status

Coverage increased (+0.4%) to 78.552% when pulling 729df6c92ff06336199437fced8ae83ab1c96263 on adlai:merge-policies into 46fe3e8bbe9bbc769db1732029e90db9591fe8d5 on JoinMarket-Org:develop.

coveralls avatar Oct 03 '17 09:10 coveralls