We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Polynomial
Vec<E::Fr>
Operations over polynomial and commitment would be changed because of the new type definition.
zero-chain/core/sonic/src/polynomials/mod.rs
Line 6 in d866d2e
The text was updated successfully, but these errors were encountered:
It could be type alias.
Sorry, something went wrong.
You might want +, *, etc. defined, which prevents using a type alias. You might check into using both Vec and SmallVec with roughly:
+
*
Vec
SmallVec
pub struct Polynomial<E,V>(V) where E: Engine: V: Borrow<[E::Fr]>+BorrowMut<[E::Fr]>+Default;
Appears VecLike was deprecated in favor of just Extend though, so maybe mutation does not work so well.
VecLike
Extend
LGTM! It should be implemented as operator overloadings. I've never used SmallVec. How much would it be optimized actually?
No branches or pull requests
Operations over polynomial and commitment would be changed because of the new type definition.
zero-chain/core/sonic/src/polynomials/mod.rs
Line 6 in d866d2e
The text was updated successfully, but these errors were encountered: