Skip to content

feat(RingTheory/Extension): naive cotangent complex commutes with flat base change#35593

Open
chrisflav wants to merge 21 commits intoleanprover-community:masterfrom
chrisflav:cotangent-basechange.2
Open

feat(RingTheory/Extension): naive cotangent complex commutes with flat base change#35593
chrisflav wants to merge 21 commits intoleanprover-community:masterfrom
chrisflav:cotangent-basechange.2

Conversation

@chrisflav
Copy link
Copy Markdown
Member

@chrisflav chrisflav commented Feb 21, 2026

@chrisflav chrisflav added the t-ring-theory Ring theory label Feb 21, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 21, 2026

PR summary 11476c36a7

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.RingTheory.Extension.Cotangent.BaseChange 1606 1621 +15 (+0.93%)
Mathlib.RingTheory.Extension.Generators 1568 1570 +2 (+0.13%)
Import changes for all files
Files Import difference
Mathlib.RingTheory.Extension.Generators 2
Mathlib.RingTheory.Extension.Cotangent.BaseChange 15

Declarations diff

+ Hom.ofAlgHom
+ Hom.toAlgHom_ofAlgHom
+ baseChangeFromBaseChange
+ baseChangeFromBaseChange_apply
+ baseChangeToBaseChange
+ baseChangeToBaseChange_apply
+ cotangentEquivCotangentKer
+ equivOfEq
+ equivOfEq_symm
+ equivOfEq_toCotangent
+ ker_baseChange
+ lift
+ lift_comp_toCotangent
+ lift_surjective_iff
+ lift_toCotangent
+ tensorCotangentOfFlat
+ tensorCotangentOfFlat_tmul
+ tensorH1CotangentOfFlat_tmul
+ tensorToH1Cotangent
+ tensorToH1Cotangent_bijective_of_flat
+ tensorToH1Cotangent_tmul
++ tensorH1CotangentOfFlat
- Cotangent.lift
- Cotangent.lift_comp_toCotangent
- Cotangent.lift_surjective_iff
- Cotangent.lift_toCotangent

You can run this locally as follows
## summary with just the declaration names:
./scripts/pr_summary/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/pr_summary/declarations_diff.sh long <optional_commit>

The doc-module for scripts/pr_summary/declarations_diff.sh contains some details about this script.


Increase in tech debt: (relative, absolute) = (3.00, 0.00)
Current number Change Type
6900 3 backward.isDefEq.respectTransparency

Current commit ac8114c20f
Reference commit 11476c36a7

You can run this locally as

./scripts/reporting/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@mathlib-dependent-issues mathlib-dependent-issues bot added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Feb 21, 2026
@mathlib-dependent-issues mathlib-dependent-issues bot removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Mar 3, 2026
@mathlib-merge-conflicts mathlib-merge-conflicts bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 3, 2026
@mathlib-merge-conflicts
Copy link
Copy Markdown

This pull request has conflicts, please merge master and resolve them.

@mathlib-dependent-issues mathlib-dependent-issues bot added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Mar 4, 2026
@mathlib-dependent-issues mathlib-dependent-issues bot removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Mar 17, 2026
@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 17, 2026
0 0
(h1Cotangentι.restrictScalars R)
((P.baseChange (T := T)).cotangentComplex.restrictScalars R)
0 0
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
0 0
-- The row ...
0 0

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is not another row, but the vertical maps. I have added a comment, please let me know if it is not clear enough.

@mariainesdff mariainesdff added the awaiting-author A reviewer has asked the author a question or requested changes. label Mar 31, 2026
chrisflav and others added 6 commits April 1, 2026 16:04
@chrisflav
Copy link
Copy Markdown
Member Author

Thanks for the review @mariainesdff!

@chrisflav chrisflav added awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. and removed awaiting-author A reviewer has asked the author a question or requested changes. awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. labels Apr 1, 2026
Copy link
Copy Markdown
Contributor

@robin-carlier robin-carlier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only a minor question, I’m happy with the rest

end Extension

/-- Flat base change commutes with `H1Cotangent`. -/
noncomputable def tensorH1CotangentOfFlat (T : Type*) [CommRing T] [Algebra R T] [Module.Flat R T] :
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we hope for somewhat reasonable lemmas characterizing this def?

simp [RingHom.algebraMap_toAlgebra]

@[simp]
lemma baseChangeFromBaseChange_apply (x) :
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
lemma baseChangeFromBaseChange_apply (x) :
lemma baseChangeFromBaseChange_apply (x : P.toExtension.baseChange.Ring) :


set_option backward.isDefEq.respectTransparency false in
@[simp]
lemma baseChangeToBaseChange_apply (x) :
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
lemma baseChangeToBaseChange_apply (x) :
lemma baseChangeToBaseChange_apply (x : (baseChange T P).toExtension.Ring) :

@robin-carlier robin-carlier added the awaiting-author A reviewer has asked the author a question or requested changes. label Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-author A reviewer has asked the author a question or requested changes. t-ring-theory Ring theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants