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

ADVZ PayloadProver intelligently use Small or Large range proof #441

Open
ggutoski opened this issue Dec 5, 2023 · 0 comments
Open

ADVZ PayloadProver intelligently use Small or Large range proof #441

ggutoski opened this issue Dec 5, 2023 · 0 comments
Assignees

Comments

@ggutoski
Copy link
Contributor

ggutoski commented Dec 5, 2023

Based on Originally posted by @ggutoski in #438 (comment)

Problem

SmallRangeProof is strictly worse than LargeRangeProof if the given range is large. We should make it easy for the user to avoid misuse of SmallRangeProof.

Proposed solution

Add another impl of PayloadProver that intelligently selects the proof type based on the user's input. The return type would be an enum with Small and Large variants. The existing impls for Small and Large remain as-is so that the user retains the ability to enforce a preference for Small or Large.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant