Refactoring CompPoly Out of Arklib#307
Conversation
|
@alexanderlhicks @quangvdao @chung-thai-nguyen - have I missed anything in Arklib that has already been ported but isn't removed in this PR? We're not currently counting field specs, waiting for the version bump to be merged to do those. |
🤖 Gemini PR SummaryThis PR represents a major architectural shift to decouple core mathematical foundations from RefactoringThis is the primary focus of the PR, involving the wholesale removal of core modules and the redirection of internal dependencies.
Features
Documentation
Fixes
Analysis of Changes
✅ **Removed:** 15 `sorry`(s)
🎨 **Style Guide Adherence**All changes adhere to the style guide. 📄 **Per-File Summaries**
Last updated: 2026-02-18 17:28 UTC. |
lakefile.toml
Outdated
| [[require]] | ||
| name = "CompPoly" | ||
| git = "https://github.com/Verified-zkEVM/CompPoly" | ||
| rev = "f1530fe653f426b6477e3e00253d7336736afa09" |
There was a problem hiding this comment.
probably best for legibility to put a tag rather than hash here
There was a problem hiding this comment.
good call, I added the tag v4.26.0-patch1 to CompPoly and have updated this now to rev "v4.26.0-patch1"
d472625 to
189992b
Compare
|
NB Field specs were also ported into this PR and have been merged. Ready to merge |
Assuming that we merge #303 (although this will likely work no matter the 4.26.0 bump):
In an effort to have Arklib depend on CompPoly, this pull request removes the following files that were ported over to CompPoly:
This pull request corresponds to the pull request on CompPoly that ports over changes to the CompPoly files done in Arklib since they were first ported. We currently pin the version of CompPoly for Arklib to the latest commit en lieu of pinning it to 4.26.0 bc these changes are currently needed to get the dependencies working. For future version bumps, we can coordinate releases of CompPoly with those of Arklib.