Skip to content

Handle T_1 in supersingular Hecke matrices - #42684

Open
OutisNemosseus wants to merge 2 commits into
sagemath:developfrom
OutisNemosseus:ssmod-t1-fix
Open

Handle T_1 in supersingular Hecke matrices#42684
OutisNemosseus wants to merge 2 commits into
sagemath:developfrom
OutisNemosseus:ssmod-t1-fix

Conversation

@OutisNemosseus

@OutisNemosseus OutisNemosseus commented Aug 17, 2026

Copy link
Copy Markdown

Description

The generic Hecke framework treats T_1 as the identity, but
SupersingularModule.hecke_matrix() overrides the generic implementation
and does not handle L = 1.

As a result,

SupersingularModule.hecke_matrix(1)
-> Phi_polys(1, ...)
-> pari.polmodular(1)
-> PariError: domain error in polmodular_ZM: L < 2

This adds the missing L == 1 identity case and a regression doctest.

Testing

The modified ssmod.py was loaded directly from this branch and checked with:

T1 =
[1 0 0]
[0 1 0]
[0 0 1]

T1 identity: PASS
T2 control: PASS

git diff --check and git show --check pass.

@OutisNemosseus

Copy link
Copy Markdown
Author

@github-actions

Copy link
Copy Markdown

Documentation preview for this PR (built with commit 3ec283d; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

@OutisNemosseus
OutisNemosseus marked this pull request as ready for review August 23, 2026 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants