LMFDB / lmfdb

L-Functions and Modular Forms Database
Other
246 stars 199 forks source link

dq to rcs #2946

Closed jvoight closed 5 years ago

jvoight commented 5 years ago

Rename dq knowls according to the convention: rcs.rigor.xyz (where xyz = hmf, ec, ecnf, etc.) rcs.cande.xyz rcs.source.xyz

roed314 commented 5 years ago

The following renames don't work since the new knowl already exists. What should happen for each of these?

dq.artin.extent -> rcs.cande.artin dq.character.dirichlet.extent -> rcs.cande.character.dirichlet dq.gg.extent -> rcs.cande.gg dq.lf.extent -> rcs.cande.lf dq.nf.extent -> rcs.cande.nf dq.zeros.zeta.extent -> rcs.cande.zeros.zeta

jwj61 commented 5 years ago

With artin, lf, gg, and nf, the dq knowl is used for one thing, and the cande for something else. So, don't change them.

It looks like rcs.cande.character.dirichlet should be rcs.source.character.dirichlet and dq.character.dirichlet.extent should be rcs.cande.character.dirichlet .

AndrewVSutherland commented 5 years ago

I think the dq.artin/gg/lf/nf.extent knowls should be renamed to artin/gg/lf/nf.extent (i.e. remove the dq prefix), and I agree that the Dirichlet character knowls should be renamed as @jwj61 suggests.

AndrewVSutherland commented 5 years ago

The rcs.cande.character.dirichlet knowl was renamed to rcs.source.character.dirichlet in PR #3042.

jwj61 commented 5 years ago

Reasonable change for artin/gg/lf/nf

jwj61 commented 5 years ago

I started the process. In this pass, I will also rename the knowl for gap numbers of small groups (currently under top level curve).

jwj61 commented 5 years ago

These have all been renamed.