feat(coding-theory): integrate CompPoly's executable Guruswami-Sudan decoder - #574
feat(coding-theory): integrate CompPoly's executable Guruswami-Sudan decoder#574olympichek wants to merge 7 commits into
Conversation
🤖 PR SummaryMathematical Formalization
Executable Integration
Refactoring and Infrastructure
Statistics
Lean Declarations ✏️ **Added:** 47 declaration(s)
📋 **Additional Analysis**Pull Request Guidelines
Style and Naming Guidelines
Syntax and Formatting
Documentation Standards
📄 **Per-File Summaries**
Last updated: 2026-06-16 14:12 UTC. |
2b7f528 to
f345571
Compare
|
@MavenRain thank you for your review of this PR! I just pushed a pair of new commits to address issues/gaps you pointed out |
|
Coordination note (complementary lane, no file overlap intended): we are preparing the encoder-side executable bridge — a theorem that CompPoly's Two touch points with this PR:
Placement guidance welcome ( The Institute for Ontological Mathematics (IAOM) / Equation Capital dba Apoth3osis. |
|
This is showing as conflicting, so I resolved it locally and build-verified the result — offering the resolution here since we contributed the sibling decoder work in CompPoly (#267) and know that side of the dependency. The conflict is narrower than it looks. Only two files actually conflict:
So the resolution is to take It survives, with one repair. After taking
- simpa using (List.nodup_ofFn.2 ωs.injective)
+ simpa [Function.comp_def] using (List.nodup_ofFn.2 ωs.injective)That is the whole repair — nothing mathematical changed, and Verified: merge Happy to open a PR against The Institute for Ontological Mathematics (IAOM) / Equation Capital dba Apoth3osis. |
`simp` no longer unfolds the point-projection composition to its pointwise form, so `distinctXCoordinates_of_represents` needs `Function.comp_def` to close. Also switch `Executable.lean` to `Mathlib.Analysis.Real.Sqrt`, matching the sibling GS files; `Mathlib.Data.Real.Sqrt` is now deprecated. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
af4179b to
077d153
Compare
🤖 PR SummaryThis PR integrates ArkLib's existing Guruswami–Sudan specification with CompPoly's executable decoder, establishing a certified translation layer that lets the concrete decoder run on ArkLib Reed–Solomon inputs and proves its output matches the previously purely existential degree-and-distance specification. Mathematical Formalization
Protocols / Soundness
Infrastructure / CI
Refactoring
Note: The PR body describes a small wrapper and a bounded search; the actual substance is concentrated in Statistics
Lean Declarations ✏️ Added: 47 declaration(s)
📋 **Additional Analysis**No findings. 📄 **Per-File Summaries**
Last updated: 2026-07-28 16:35 UTC. |
I just rebased the branch and applied your suggestions |
Overview
This PR connects ArkLib's Guruswami-Sudan development to the executable decoder recently added in CompPoly. It adds the translation layer needed to run the CompPoly decoder on ArkLib Reed-Solomon inputs and state the result in ArkLib's existing degree-and-distance specification.
This PR includes:
v4.30.0-patch1, together with the new imports.Correctness
The main theorem says that the executable output matches the ArkLib specification: returned polynomials have degree
< kand are within the requested Hamming radius, and every polynomial with those properties appears in the output. There are also compatibility lemmas relating this statement to ArkLib's existingdecoderunder the Johnson-radius assumption.