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.

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

SOT-133: Implement Plookup #11

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

SOT-133: Implement Plookup #11

wants to merge 6 commits into from

Conversation

hduoc2003
Copy link
Contributor

No description provided.

@hduoc2003 hduoc2003 self-assigned this Apr 23, 2024
@linear Linear
Copy link

linear bot commented Apr 23, 2024

SOT-133 Implement Plookup

github-advanced-security[bot]
github-advanced-security bot found potential problems Apr 23, 2024
View reviewed changes
lookup/src/plookup/lookup_vector.rs Fixed Show fixed Hide fixed
lookup/src/transcript.rs Fixed Show fixed Hide fixed
lookup/src/transcript.rs Fixed Show fixed Hide fixed
lookup/src/plookup/lookup_vector.rs Fixed Show fixed Hide fixed
lookup/src/plookup/lookup_vector.rs Fixed Show fixed Hide fixed
lookup/src/plookup/lookup_vector.rs Fixed Show fixed Hide fixed
lookup/src/multiset/mod.rs Fixed Show fixed Hide fixed
lookup/src/multiset/multiset.rs Fixed Show fixed Hide fixed
lookup/src/utils/fold.rs Fixed Show fixed Hide fixed
lookup/src/types.rs Fixed Show fixed Hide fixed
VanhGer
VanhGer reviewed May 20, 2024
View reviewed changes
lookup/src/errors.rs Outdated Show resolved Hide resolved
VanhGer
VanhGer reviewed May 20, 2024
View reviewed changes
lookup/src/lookup.rs Outdated Show resolved Hide resolved
VanhGer
VanhGer reviewed May 20, 2024
View reviewed changes
lookup/src/multiset.rs Outdated Show resolved Hide resolved
VanhGer
VanhGer reviewed May 20, 2024
View reviewed changes
lookup/src/pcs/kzg10.rs Outdated Show resolved Hide resolved
@VanhGer
Copy link
Contributor

VanhGer commented May 20, 2024

You should provide an example usage for your library.

@VanhGer VanhGer requested a review from zk-steve May 20, 2024 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VanhGer VanhGer VanhGer left review comments

@zk-steve zk-steve Awaiting requested review from zk-steve

At least 1 approving review is required to merge this pull request.

Assignees

@hduoc2003 hduoc2003

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hduoc2003 @VanhGer