celestiaorg / celestiaorg/celestia-node

full/availability: Vet parameters in reconstruction tests

Open
#2,779 1 comment 0 reactions 1 assignee Claimed by @adlerjohn View on GitHub
needs:triage
Dominant language
Go
Stars
996
Forks
1.1k
Avg merge
1d 6h
Merged PRs (30d)
34

Description

https://github.com/celestiaorg/celestia-node/pull/2697 reworked our reconstruction tests to involve worst-case data withholding scenarios. This required us to dive deeper into the `k`, `c`, and `s` parameters in [the original DA paper](https://arxiv.org/pdf/1809.09044.pdf). As per @distractedm1nd, the `c` parameter does not represent the withholding case, so he made a [custom script](https://gist.github.com/distractedm1nd/d2b306da13581b97bbbcefaa5b050738) that simulates reconstruction and reevaluates `c` for withholding scenario.

However, we need more confidence in the revaluation results and to vet and analyze them more.

cc @musalbas

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.