[unvetted AI slop] Use the Goppa polynomial base field - #814
[unvetted AI slop] Use the Goppa polynomial base field#814Krastanov-agent wants to merge 1 commit into
Conversation
Why this fix is necessaryFor a binary Goppa code, the polynomial and support must belong to the same concrete field: The previous three-argument constructor discarded the field attached to As a result, the old constructor could create The authoritative field is already available as Why the existing tests did not catch itMost Goppa coverage bypassed this code path:
The two existing examples that did use the default-support constructor created their polynomial field with the same parent(a) === base_ring(parent(g))for the generated support. The regression in this PR checks that invariant directly. This matters even when two field presentations happen to produce the same numerical code parameters: the returned |
Summary\n\n- build the default support from the Goppa polynomial's actual base field\n- add a regression using a noncanonical finite-field representation\n\n## Testing\n\nNot run locally; relying entirely on repository CI as requested.