New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Merged by Bors] - feat: changeLevel for Dirichlet characters #7263
Conversation
mo271
commented
Sep 19, 2023
@MichaelStollBayreuth adding you here, because of the helpful suggestions in #7010. |
Can we add some "obvious" lemmas about change level here, like |
I was going to do that in a follow-up pull request in order to keep them small, but I'm happy to do it here now. Added those. |
I appreciate that! I think there is some sort of happy medium around 100 lines for a new Def + the basic lemmas to be sure that the Def works as intended. |
Co-authored-by: Alex J Best <alex.j.best@gmail.com>
Co-authored-by: Alex J Best <alex.j.best@gmail.com>
Should the indentation in lemma toUnitHom_eq_char' {a : ZMod n} (ha : IsUnit a) :
χ a = χ.toUnitHom ha.unit := by simp
lemma toUnitHom_eq_iff (ψ : DirichletCharacter R n) :
toUnitHom χ = toUnitHom ψ ↔ χ = ψ := by simp
lemma eval_modulus_sub (x : ZMod n) :
χ (n - x) = χ (-x) := by simp also be changed (to indent by 4 for statements on the line(s) after ":")? |
I just put them on one line. |
Looks good to me now! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maintainer merge
🚀 Pull request has been placed on the maintainer queue by alexjbest. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 🎉
bors merge
Co-authored-by: Moritz Firsching <firsching@google.com>
Pull request successfully merged into master. Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
Co-authored-by: Moritz Firsching <firsching@google.com>