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

Changes from July 26 meeting #31

Merged
merged 7 commits into from
Aug 2, 2024
Merged

Conversation

daira
Copy link
Collaborator

@daira daira commented Jul 26, 2024

  • Use $\rightarrow$ instead of $\mapsto$ to indicate function types.
  • Consistent use of $\mathrel{⦂}$.
  • In the definition of a multivariate polynomial, $c_z$ should not be zero.
  • Define $m'_f$ and require fixed concrete columns to be on the left.
  • Add a forward reference from the table describing instances to the section that defines multivariate polynomials.
  • Spelling exceptions for the LTeX vscode extension.
  • Add hints as an explicit argument to ok_for.

Signed-off-by: Daira-Emma Hopwood <[email protected]>
…ero.

(If it could be zero then the definition of "might be used" in
optimizations.md would be unnecessarily conservative, because the
variables appearing with non-zero exponents in a term with coefficient
zero are effectively unused.)

Signed-off-by: Daira-Emma Hopwood <[email protected]>
This fixes a bug in the definition of the matrix of fixed concrete
columns $f'$.

Signed-off-by: Daira-Emma Hopwood <[email protected]>
section that defines multivariate polynomials.

Signed-off-by: Daira-Emma Hopwood <[email protected]>
@daira daira requested a review from str4d July 26, 2024 16:33
extension is not installed).

Signed-off-by: Daira-Emma Hopwood <[email protected]>
@daira daira requested review from mmaller and removed request for str4d August 2, 2024 13:24
@daira daira merged commit 9d68a15 into zkpstandard:main Aug 2, 2024
8 checks passed
@daira daira deleted the meeting-july-26 branch August 2, 2024 13:25
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