-
Notifications
You must be signed in to change notification settings - Fork 265
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
refactor: Renaming around Protogalaxy Prover (#8272)
This is literally just a ton of renaming + shuffling some declaration and defs in the Protogalaxy prover so the orders of these match. Some highlights: - I wanted to stop using the term "optimised" around the Pg optimization that skips computing zero because we have many optimizations and the name was unclear. I also put the extra qualifier on the non-production case where we _don't_ use that optimization, which is currently just in tests of the combiner. - Ariel told me that the real name of the protocol is Protogalaxy--this is in the name of the paper in pdf form, but not on the eprint page :shrug: - `PowPolynomial` is now a misnomer because it doesn't involve powers of a challenge $\beta$, but rather a bunch of monomials generate from a set of $\beta_i$'s. So I rename this and the corresponding files. I didn't have to change this but I do think clarity here comes at a low enough cost to merit making the change. - We have functions that refer to the "full honk" relation, which is fine, but strictly speaking there is no connection to Honk when using Protogalaxy or sumcheck in insolation, so it makes sense to give more agnostic and IMO slightly clearer names.
- Loading branch information
1 parent
9ee8dd6
commit be2169d
Showing
40 changed files
with
395 additions
and
388 deletions.
There are no files selected for viewing
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
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
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
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
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
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
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
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
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
Oops, something went wrong.