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

Further clarify proofs of invalid erasure coding #1134

Open
4 tasks
liamsi opened this issue May 31, 2021 · 5 comments
Open
4 tasks

Further clarify proofs of invalid erasure coding #1134

liamsi opened this issue May 31, 2021 · 5 comments
Labels
kind:docs For solely documentation PRs

Comments

@liamsi
Copy link
Member

liamsi commented May 31, 2021

Summary

The node types section only states who (which node type) is able to generate proofs of invalid erasure coding. Nowhere is explained what happens after generating them. This leaves a lot of room for interpretation. Who should care about those proofs and how will they be propagated (to who)?

Consumers of fraud proofs would be anyone that just does DAS. I guess that could be made more explicit.

Also, do they trigger slashable events? If yes, who will be slashed?

In terms of what penalities fraud proofs are associated with, that's more of a consensus/evidence concern.

Action Items

These should be separate issues that should be handled successively:

  • specify the data structure for these fraud proofs
  • specify the flow of events and the underlying assumptions
    • clarify grace period to light clients to wait for erasure coding fraud proofs and implications for light clients (wait for fraud proofs?)
    • specify penalties / slashing (if any)

related issue about evidence types: celestiaorg/celestia-specs#23
also related: celestiaorg/celestia-specs#110

@liamsi
Copy link
Member Author

liamsi commented May 31, 2021

I've put this in the MVP project board. Although we don't need this fro implementing the MVP, this should be clarified in all detail before we start the implementation (devnet).

@adlerjohn adlerjohn transferred this issue from celestiaorg/celestia-specs Sep 19, 2022
@Wondertan
Copy link
Member

Do I understand correctly that this is just a spec/docs issue?

@liamsi
Copy link
Member Author

liamsi commented Sep 23, 2022

yes

@Bidon15
Copy link
Member

Bidon15 commented Nov 1, 2022

Grooming 01/11/2022:

@liamsi from node side, we have already got an ADR and implementation in the repo
Still, docs side we have no info, so is it relevant to pass this issue on docs explanation to devRel? wdyt?

@Wondertan
Copy link
Member

Wondertan commented Nov 11, 2022

@liamsi, bong

@ramin ramin added the kind:docs For solely documentation PRs label Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:docs For solely documentation PRs
Projects
None yet
Development

No branches or pull requests

4 participants