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

TIP-19 Dust Protection Based on Byte Costs #39

Merged
merged 39 commits into from
Jul 13, 2022
Merged

Conversation

muXxer
Copy link
Contributor

@muXxer muXxer commented Nov 4, 2021

This RFC describes a new dust protection mechanism that prevents bloating of the IOTA ledger database.

Rendered Version

The scripts for byte cost calculations can be found here.

@luca-moser
Copy link
Member

luca-moser commented Nov 19, 2021

Minimum state rent virtual byte counts are incorrect given that the calculations still work with uint16 for denoting the feature blocks slice length (instead of uint8).

@lzpap lzpap changed the title RFC: Dust Protection based on byte costs TIP-19 Dust Protection Based on Byte Costs Dec 15, 2021
@oopsmonk
Copy link

<tr>
<th>Name</th>
<th>Description</th>
<th>Weight</th>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should specify that these can be floating point / fractional values. vbyte_cost on the other hand is of type integer, right? Maybe we should specify that somewhere too?

tips/TIP-0019/tip-0019.md Outdated Show resolved Hide resolved
tips/TIP-0019/tip-0019.md Outdated Show resolved Hide resolved
tips/TIP-0019/tip-0019.md Outdated Show resolved Hide resolved
tips/TIP-0019/tip-0019.md Outdated Show resolved Hide resolved
tips/TIP-0019/tip-0019.md Outdated Show resolved Hide resolved
tips/TIP-0019/tip-0019.md Outdated Show resolved Hide resolved
tips/TIP-0019/tip-0019.md Outdated Show resolved Hide resolved
tips/TIP-0019/tip-0019.md Outdated Show resolved Hide resolved
tips/TIP-0019/tip-0019.md Outdated Show resolved Hide resolved

Another solution is to convert all `SigLockedDustAllowanceOutput` into `BasicOutputs` and leave the `SigLockedSingleOutput` below 1 MIOTA untouched.

### Unresolved questions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When consider merging this TIP, we should remove this section.
Are there even still open? I guess we obviously have to define some v_byte_cost value so we could maybe just remove the entire section.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should remove this section as you said because any parameter values are subject to be put into a separate TIP or the actual network configuration anyway, so this is not the place to put it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will removed it 👍

lzpap and others added 19 commits July 11, 2022 10:54
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
Co-authored-by: Wolfgang Welz <welzwo@gmail.com>
@lzpap lzpap merged commit 00adc89 into iotaledger:main Jul 13, 2022
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.

7 participants