-
Notifications
You must be signed in to change notification settings - Fork 387
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
Potentially increase share size #732
Labels
specs
directly relevant to the specs
Comments
To provide some additional context: since
it makes sense to make the share size larger rather than smaller. |
The main drawback of increasing the chunk size is that it makes data availability sampling more expensive, but we should do some calculations on this. |
Related: #711 |
36 tasks
Closed by #850 |
Repository owner
moved this from TODO
to Done
in Celestia Node
Oct 31, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We currently set the share size (rather arbitrarily) to 256 bytes. IMO this should be increased to maybe 1KB or more. Independent of what I think there must be a rationale document for the parameter choice explaining pros/cons and why we settled on whatever value.
The text was updated successfully, but these errors were encountered: