search

LMFDB / lmfdb

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

Specific knowls to replace #3014

Closed davidfarmer closed 5 years ago

davidfarmer commented 5 years ago

This is related to issue #2992 , but here I am listing pairs of knowls which both exist but one of them should be deleted and replaced by another. Se that issue to understand the possible pitfalls. In particular, the content of the knowl which is to be deleted will actually survive, so you have to first copy the good material into the bad knowl.

I am assigning myself to do this, but will not object if someone else does it first!

Feel free to add to this list.

Delete: mf.maass.mwf.fricke Replace by: cmf.fricke

Delete: mf.maass.mwf.atkinlehner Replace by: cmf.atkin-lehner

Delete: lfunction.level Replace by: lfunction.conductor

pascalmolin commented 5 years ago

Added to this list: Delete character.dirichlet.induced

I have already removed all references and redirected them to the dupplicate character.dirichlet.induce

In my opinion, the content of character.dirichlet.induced can be discarded (I do not think the extra details are important in any context), but someone may disagree.

AndrewVSutherland commented 5 years ago

We have automated tools that make this very easy, I just made the changes, see PR #3015

AndrewVSutherland commented 5 years ago

@pascalmolin I just deleted character.dirichlet.induced (safe since it is not referenced anywhere, and I agree the additional content was superfluous).

@davidfarmer if you could just sanity check and then merge PR #3015, I will then push to dev/web, delete the knowls, and then close this issue.

davidfarmer commented 5 years ago

I will check and merge. I want to make sure that all the correct content is in the knowls that will remain.

On Tue, 30 Apr 2019, Andrew Sutherland wrote:

@pascalmolin I just deleted character.dirichlet.induced (safe since it is not referenced anywhere, and I agree the additional content was superfluous).

@davidfarmer if you could just sanity check and then merge PR #3015, I will then push to dev/web, delete the knowls, and then close this issue.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.[AABTULGKX32ICLF7YYCTIS3PTCRARANCNFSM4HJO2UPQ.gif]

AndrewVSutherland commented 5 years ago

The code with the updated knowl links is now in production and the knowls mf.maass.mwf.fricke, mf.maass.mwf.atkinlehner, lfunction.level, character.dirichlet.induced have been deleted.

Closing.