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

Add verification check between precompute data and vid commitment #560

Open
akonring opened this issue Apr 18, 2024 · 1 comment · May be fixed by #561
Open

Add verification check between precompute data and vid commitment #560

akonring opened this issue Apr 18, 2024 · 1 comment · May be fixed by #561
Labels
optimize-vid https://www.notion.so/espressosys/9d835f79d4504926b8b3bb3d015abf06?v=b7028cdaea804b7aa918af95c0cd651 vid

Comments

@akonring
Copy link
Contributor

akonring commented Apr 18, 2024

Precomputable trait is designed to allow for data (currently only kzg batch commitment) to be reused in subsequent call to disperse.

Since the builder cannot be trusted to use the same kzg batch commitments inside precomputed data (as the output of commit_only_precompute) we should add a mechanism to ensure that these are the same.

@akonring akonring added vid optimize-vid https://www.notion.so/espressosys/9d835f79d4504926b8b3bb3d015abf06?v=b7028cdaea804b7aa918af95c0cd651 labels Apr 18, 2024
@ggutoski
Copy link
Contributor

The goal is to make something analogous to

fn VidScheme::is_consistent(commit: &Self::Commit, common: &Self::Common) -> VidResult<()>

@akonring akonring linked a pull request Apr 19, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
optimize-vid https://www.notion.so/espressosys/9d835f79d4504926b8b3bb3d015abf06?v=b7028cdaea804b7aa918af95c0cd651 vid
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants