Conversation
🤖 Gemini PR SummaryThis pull request performs a major import of field specifications and algebraic constructions from Features
Refactoring
Documentation
Analysis of Changes
❌ **Added:** 12 `sorry`(s)
🎨 **Style Guide Adherence**This review identifies violations of the CompPoly Style Guide found in the provided code changes. CompPoly/Data/Classes/DCast.lean
CompPoly/Data/Polynomial/Frobenius.lean
CompPoly/Data/Polynomial/MonomialBasis.lean
CompPoly/Data/RingTheory/AlgebraTower.lean
CompPoly/Fields/AdditiveNTT/AdditiveNTT.lean
CompPoly/Fields/AdditiveNTT/NovelPolynomialBasis.lean
📄 **Per-File Summaries**
Last updated: 2026-02-17 17:27 UTC. |
|
@alexanderlhicks @Julek @quangvdao in porting in the field specifications from Arklib I've opted to have Fields in their own directory, and to keep all the common (nonbinary) fields in the |
This pull request imports field specs from Arklib, starting with nonbinary fields. Instead of differentiating between Binary and non-Binary fields in the file structure, I've opted to default to "Fields" referring to the nonbinary fields. Binary fields are in their own subdirectory, as well as AdditiveNTT.
Corresponds to a sister pull request: Verified-zkEVM/ArkLib#309