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

Fold the non_hiding_kzg module in relevant files #328

Merged
merged 2 commits into from
Feb 19, 2024

Conversation

huitseeker
Copy link
Member

@huitseeker huitseeker commented Feb 17, 2024

Moves the contents of Univariate KZG in two locations:

  • kzg_commitments, for those types needed there (essentially setting up the public parameters),
  • non_hiding_zeromorph, for the parts specifically used by ZM (and not yet by HyperKZG/Shplonk)

@huitseeker huitseeker changed the title Revert "feat: Implement batch operations in non_hiding_kzg module (#269)" Revert "feat: Implement batch operations in non_hiding_kzg module (#269)" (easy) Feb 19, 2024
@huitseeker huitseeker changed the title Revert "feat: Implement batch operations in non_hiding_kzg module (#269)" (easy) Fold the non_hiding_kzg module in relevant files Feb 19, 2024
…eed it

- Moved `UniversalParams` and several dependent structures for the KZG10 scheme in the `kzg_commitment.rs` file.
- Deleted the `non_hiding_kzg.rs` file,
- Consolidated KZG related structs under the `kzg_commitment` module,
- Updated `mod.rs` to reflect the removal of the `non_hiding_kzg` module.
@storojs72
Copy link
Member

LGTM, Thanks!

@huitseeker huitseeker added this pull request to the merge queue Feb 19, 2024
Merged via the queue into argumentcomputer:dev with commit e1a69d4 Feb 19, 2024
9 checks passed
@huitseeker huitseeker deleted the remove_batch branch February 19, 2024 17:09
github-actions bot pushed a commit that referenced this pull request Aug 19, 2024
* optimize computation of cross-term

* clippy fixes

* cargo fmt
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

Successfully merging this pull request may close these issues.

2 participants