LMFDB / lmfdb

L-Functions and Modular Forms Database
Other
249 stars 200 forks source link

Magma command for modular form does not work #5432

Open jwj61 opened 1 year ago

jwj61 commented 1 year ago

I the magma file for downloading the form 633.1.m.b, it suggests the command "MakeCharacter_633_m_Hecke();", but typing this (after loading the file) gives an error.

edgarcosta commented 1 year ago

Perhaps Kf shouldn't be an argument, otherwise we need to suggest

> MakeCharacter_633_m_Hecke(NumberField(Polynomial([1,0,-1,0,1,0,-1,0,1])));
$.1*$.2^2
JohnCremona commented 1 year ago

5397 was created to try to avoid this sort of thing

edgarcosta commented 1 year ago

I don't follow. We are already testing for code necessary to generate the newform: https://github.com/LMFDB/lmfdb/blob/master/lmfdb/classical_modular_forms/test_cmf2.py#L71-L115 However, we don't test this optional function. Once we fix it, one can add a check for this function, but we still need to do this manually.