[Merged by Bors] - feat: add Mathlib.RingTheory.RootsOfUnity.CyclotomicUnits#30085
[Merged by Bors] - feat: add Mathlib.RingTheory.RootsOfUnity.CyclotomicUnits#30085riccardobrasca wants to merge 17 commits intoleanprover-community:masterfrom
Conversation
PR summary 34812cd2a7Import changes for modified filesNo significant changes to the import graph Import changes for all files
|
faenuccio
left a comment
There was a problem hiding this comment.
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) hjCo-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 |
We add basic results about cyclotomic units. From `flt-regular`.
|
Pull request successfully merged into master. Build succeeded: |
…-community#30085) We add basic results about cyclotomic units. From `flt-regular`.
…-community#30085) We add basic results about cyclotomic units. From `flt-regular`.
We add basic results about cyclotomic units.
From
flt-regular.