zcash icon indicating copy to clipboard operation
zcash copied to clipboard

Remove "dust" lower limit on txout values

Open daira opened this issue 7 years ago • 18 comments

fixes #2133

daira avatar Mar 02 '17 04:03 daira

@zkbot try

daira avatar Mar 02 '17 04:03 daira

:hourglass: Trying commit c190e7d with merge e03f6ec...

zkbot avatar Mar 02 '17 04:03 zkbot

:broken_heart: Test failed - zcash

zkbot avatar Mar 02 '17 05:03 zkbot

Fixing wallet_protectcoinbase.py now.

daira avatar Mar 03 '17 04:03 daira

@zkbot try

daira avatar Mar 03 '17 04:03 daira

:hourglass: Trying commit 96d1593 with merge fb88be6...

zkbot avatar Mar 03 '17 04:03 zkbot

:sunny: Test successful - zcash

zkbot avatar Mar 03 '17 05:03 zkbot

@daira I think this PR should be bumped. See @nathan-at-least comments: https://chat.zcashcommunity.com/channel/zcash-dev?msg=xnNmk2ywJxf4sDzTh

bitcartel avatar Mar 22 '17 22:03 bitcartel

I agree it should be bumped from 1.0.8, but please let's discuss this early in the 1.0.9 release schedule. (Note that the dust check is a standard rule and not a consensus rule.)

daira avatar Mar 23 '17 14:03 daira

Needs more discussion.

daira avatar Apr 17 '17 15:04 daira

Moving 1.0.10 tickets back into 1.0.9 since the latter has been rescheduled.

daira avatar Apr 17 '17 17:04 daira

Bumped from 1.0.9 due to time and design/evaluation costs. I believe it's safe to leave existing dust rules in for another release, and we're overcapacity for 1.0.9 already.

nathan-at-least avatar May 10 '17 19:05 nathan-at-least

:umbrella: The latest upstream changes (presumably #2440) made this pull request unmergeable. Please resolve the merge conflicts.

zkbot avatar Jun 16 '17 05:06 zkbot

Rebased onto 1.0.10.

daira avatar Jun 22 '17 19:06 daira

Note that Github is showing the commits in the wrong order; "Disable dust." should be first, followed by "Simplifications due to disabling dust."

(It thought they'd fixed that commit ordering bug since I hadn't seen it for a while. Apparently not.)

daira avatar Jun 22 '17 19:06 daira

:umbrella: The latest upstream changes (presumably #2698) made this pull request unmergeable. Please resolve the merge conflicts.

zkbot avatar Oct 31 '17 23:10 zkbot

Bump. It is my belief based on hearsay that there are no dust limits currently. I'm going to close this and if I'm wrong could someone simply reopen it. (cc @str4d & @daira )

nathan-at-least avatar Oct 27 '20 03:10 nathan-at-least

That's not correct, there are still dust limits.

daira avatar Oct 27 '20 09:10 daira

This no longer makes sense given the fee changes in ZIP 317.

sellout avatar Oct 25 '22 18:10 sellout