Skip to content

Validate proofs against data root tuple root in the DA library #165

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.

Sign up for GitHub

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

Jump to bottom
Closed
rach-id opened this issue Jul 31, 2023 · 0 comments · Fixed by #168
Closed

Validate proofs against data root tuple root in the DA library #165

rach-id opened this issue Jul 31, 2023 · 0 comments · Fixed by #168
Assignees
@rach-id
Labels
contracts

Comments

@rach-id
Copy link
Member

rach-id commented Jul 31, 2023 •
edited
Loading

After merging #161, we can implement methods that would validate separate proofs against the data root tuple root.
This can be used to verify that the square size computed is valid.
@rach-id rach-id self-assigned this Jul 31, 2023
@rach-id rach-id added this to core/app Jul 31, 2023
@rach-id rach-id moved this to In Progress in core/app Jul 31, 2023
@rach-id rach-id mentioned this issue Aug 3, 2023
Merged
5 tasks
@rach-id rach-id moved this from In Progress to In Review in core/app Aug 3, 2023
rach-id added a commit that referenced this issue Aug 4, 2023
@rach-id
feat: add intermediate proof verification in DAVerifier (#168)
c53344d 
<!--
Please read and fill out this form before submitting your PR.

Please make sure you have reviewed our contributors guide before
submitting your
first PR.
-->

## Overview

Closes #165

Also, refactors the DAVerifier test suite to use a fixture containing
all the necessary data.

## Checklist

<!-- 
Please complete the checklist to ensure that the PR is ready to be
reviewed.

IMPORTANT:
PRs should be left in Draft until the below checklist is completed.
-->

- [ ] New and updated code has appropriate documentation
- [ ] New and updated code has new and/or updated testing
- [ ] Required CI checks are passing
- [ ] Visual proof for any user facing features like CLI or
documentation updates
- [ ] Linked issues closed with keywords
@github-project-automation github-project-automation bot moved this from In Review to Done in core/app Aug 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rach-id rach-id

Labels
contracts
Projects
core/app
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add intermediate proof verification in DAVerifier rach-id/quantum-gravity-bridge
1 participant
@rach-id