-
Notifications
You must be signed in to change notification settings - Fork 840
[Merged by Bors] - feat: add Mathlib.RingTheory.RootsOfUnity.CyclotomicUnits #30085
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: add Mathlib.RingTheory.RootsOfUnity.CyclotomicUnits #30085
Conversation
PR summary 34812cd2a7Import changes for modified filesNo significant changes to the import graph Import changes for all files
|
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! Why not adding also
theorem geom_sum_isUnit' (hζ : IsPrimitiveRoot ζ n) (hj : j.Coprime n) (hj_Unit : IsUnit (j : A)) :
IsUnit (∑ i ∈ range j, ζ ^ i) := by
match n with
| 0 => simp_all
| 1 => simp_all
| n +2 => exact geom_sum_isUnit hζ (by linarith) hj
Co-authored-by: Filippo A. E. Nuccio <filippo.nuccio@univ-st-etienne.fr>
Co-authored-by: Filippo A. E. Nuccio <filippo.nuccio@univ-st-etienne.fr>
Co-authored-by: Filippo A. E. Nuccio <filippo.nuccio@univ-st-etienne.fr>
Co-authored-by: Filippo A. E. Nuccio <filippo.nuccio@univ-st-etienne.fr>
Co-authored-by: Filippo A. E. Nuccio <filippo.nuccio@univ-st-etienne.fr>
Co-authored-by: Filippo A. E. Nuccio <filippo.nuccio@univ-st-etienne.fr>
Thanks, it all LGTM. Perhaps, before merging, can you add a line of todo (perhaps taken from your FLT project) so that other contributors might know what (not) to work on? |
maintainer delegate |
🚀 Pull request has been placed on the maintainer queue by faenuccio. |
This is essentially everything we have in flt-regular, so everything else has to be done |
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
We add basic results about cyclotomic units. From `flt-regular`.
Pull request successfully merged into master. Build succeeded: |
We add basic results about cyclotomic units.
From
flt-regular
.