-
Notifications
You must be signed in to change notification settings - Fork 408
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
All Contributors Setup #120
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
LGTM, if it looks too ugly in retrospective we can consider modifying it or removing it entirely. |
danielmarinq
approved these changes
Mar 6, 2024
tess-eract
pushed a commit
that referenced
this pull request
Mar 8, 2024
* Update README.md * Update README.md * Create .all-contributorsrc
tess-eract
added a commit
that referenced
this pull request
Mar 14, 2024
* add basic setup flow using SRS * merge with main * some small fixes to bugs in the compression module from recent PRs * spartan integration fixes * tests for srs generation * sets default srs size to 19 * use `deserialize_compressed_unchecked` * remove unnecessary blinding factor from SRS generation * fix * adds SRS to git lfs * add git lfs to CI workflow * sample test srs in CI * install nexus-tools in CI * ignore tests related to large SRS * remove SRS generation from CI * merge fixes * fmt * fixes from review * small fix * remove file-manipulating tests * Adds a cli option to the prover crate for compression (#55) * testing proof deserialization * proofs save and verify * removed cargo.lock * todo: merge dorebell onto this * fixed merge mistakes * reads proof and compresses. key not yet saved to file * added back cargo.lock * moved the compression cli into prover crate * formatting * remove whitespace * added com option to prove * updated local prove * todo:save key and proof to file * derive CanonicalSerialize+CanonicalDeserialize for Spartan types * add options to save and load spartan key from file * save compressed proof to file, implement arkworks serialization * clippy * remove SRS generation from CI * ignore spartan_encode_test * integrate compression cli with recent version of nexus-tools * add cli function to sample test SRS * forgot to add new files * small compression UI fixes * add spartan setup command to main 'cargo nexus' * bump number of SRS vars to 27 * minor fix * review fixes * another round of review fixes * read pp and srs from default cache locations if unspecified during compression * add helper function to get minimum srs size for a given k --------- Co-authored-by: Dan Dore <dorebell@gmail.com> * fix broken edit links in docs (#99) * CCS implementation (#52) * Initial CCS implementation. * Remove direct construction interfaces so that everything goes through R1CS. * Trim more, Fold multipliers together, and inline satisfaction checking. * Fix formatting. * Precompute products. * Remove direct CCS construction. * Add mle helpers. * Start to integrate polynomial commitments. * Shading closer to polynomial commitments. * Initial stab at relating various polynomial types and traits. * Finish utility functions. * Fix endianness and ranges and get tests passing. * Fix formatting. * Realized there's a better way to invoke the partially fixed polynomial. * Update interfaces and some additional reworking. * Resolve clippy. * Unify shapes. * Product renaming Co-authored-by: Dan Dore <dorebell@gmail.com> * Revert "Unify shapes." This reverts commit 3463e43. * Move to polynomial/poly commitment implementations from Spartan repo. * Move to unified matrix-based model. * Fix tests. * Fix fmt. * Remove files accidently restored during rebase. * Move to using polynomial commitment exlcusively. * Fix formatting. --------- Co-authored-by: Dan Dore <dorebell@gmail.com> * All Contributors Setup (#120) * Update README.md * Update README.md * Create .all-contributorsrc * docs: add nexus-xyz as a contributor for code (#121) * docs: update README.md * docs: update .all-contributorsrc --------- Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com> * Switch Contributors (#122) * Delete .all-contributorsrc * Update README.md * fix merge conflicts --------- Co-authored-by: Guru Vamsi Policharla <guruvamsi.policharla@gmail.com> Co-authored-by: Daniel Marin <60114322+danielmarinq@users.noreply.github.com> Co-authored-by: Samuel Judson <sam@sjudson.com> Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolving #117, using the instructions from https://allcontributors.org/docs/en/bot/installation.