feat(RingTheory/Smooth): some lemmas about formally smooth#35675
feat(RingTheory/Smooth): some lemmas about formally smooth#35675Thmoas-Guan wants to merge 24 commits intoleanprover-community:masterfrom
Conversation
PR summary 6ef8cc2731Import changes for modified filesNo significant changes to the import graph Import changes for all files
|
|
There are some lemma that I didn't find an appropriate place, do anyone familiar with the field give some suggestions? |
and get rid of base change related stuffs
|
|
||
| end Algebra.TensorProduct | ||
|
|
||
| lemma Ideal.subtype_rTensor_range {R : Type*} [CommRing R] (M : Type*) [AddCommGroup M] [Module R M] |
There was a problem hiding this comment.
Can you use some variables already in scope?
|
|
||
| variable {R : Type u} [CommRing R] | ||
|
|
||
| lemma LinearMap.ker_inf_smul_top_eq_smul_of_flat {M N : Type*} [AddCommGroup M] [AddCommGroup N] |
There was a problem hiding this comment.
Can this go to another file? It is unrelated to smoothness.
There was a problem hiding this comment.
Also, maybe add a short docstring.
There was a problem hiding this comment.
@riccardobrasca do you have idea about where to move? find home suggested Mathlib.RingTheory.LocalRing.Module but I think the content is unrelated?
Also can you help have a look for good places or make private of all those lemmas separated out? Thanks.
There was a problem hiding this comment.
Something about flat modules maybe?
| /-- For flat ring homomorphism `f : R →+* S`, `I` an ideal of `R` which is square zero, | ||
| if `R ⧸ I →+* S ⧸ IS` is formally smooth, so do `f`. -/ | ||
| @[stacks 031L] | ||
| lemma Algebra.FormallySmooth.of_surjective_of_ker_eq_map_of_flat {S : Type v} [CommRing S] |
There was a problem hiding this comment.
This proof is very long. Can you try to split off as many sublemmas as possible? Don't worry if they will never be used, you can mark them as private (or even better use private/public sections).
There was a problem hiding this comment.
I'll try, I know the general convention for maintainability.
There was a problem hiding this comment.
I know it's not always easy, so don't get crazy about it, but usually it is possible to split at least a couple of results.
There was a problem hiding this comment.
I separated many annoying stuffs I remembered throughout the proof, reducing the main proof for about 50 lines but total length gaining 40.
The rest of the proof is much harder to separate because they really involve the resolution constructed in this specific case.
This PR mainly formalized the result [Stacks 031L]
This is a preliminary of Cohen Structure Theorem.