Skip to content

[Merged by Bors] - feat: covariant derivatives on vector bundles#36279

Closed
PatrickMassot wants to merge 24 commits intoleanprover-community:masterfrom
PatrickMassot:pm_covariant_derivative_basic
Closed

[Merged by Bors] - feat: covariant derivatives on vector bundles#36279
PatrickMassot wants to merge 24 commits intoleanprover-community:masterfrom
PatrickMassot:pm_covariant_derivative_basic

Conversation

@PatrickMassot
Copy link
Copy Markdown
Member

@PatrickMassot PatrickMassot commented Mar 6, 2026

From the path towards the the Levi-Civita connection and Riemannian geometry.

Co-authored-by: Heather Macbeth 25316162+hrmacbeth@users.noreply.github.com
Co-authored-by: Michael Rothgang rothgang@math.uni-bonn.de


Open in Gitpod

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 6, 2026

PR summary 151b5c4ae6

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference
Mathlib.Geometry.Manifold.VectorBundle.CovariantDerivative.Basic (new file) 2253

Declarations diff

+ ContMDiffCovariantDerivative
+ ContMDiffCovariantDerivative.affine_combination
+ ContMDiffCovariantDerivative.finite_affine_combination
+ ContMDiffCovariantDerivativeOn
+ CovariantDerivative
+ IsCovariantDerivativeOn
+ _root_.ContMDiffCovariantDerivativeOn.affine_combination
+ _root_.ContMDiffCovariantDerivativeOn.finite_affine_combination
+ addOneForm
+ add_one_form
+ congr_of_eqOn
+ congr_of_eventuallyEq
+ contMDiffCovariantDerivativeOn_univ_iff
+ differenceAux
+ differenceAux_tensorial
+ difference_apply
+ extDerivFun
+ extDerivFun_add
+ extDerivFun_zero
+ iUnion
+ instance : CoeFun (CovariantDerivative I F V)
+ isCovariantDerivativeOn
+ mono
+ of_isCovariantDerivativeOn_of_open_cover
+ of_isCovariantDerivativeOn_of_open_cover_coe
+ smulRight_zero
+ smul_const
+ zero_smulRight
++ affine_combination
++ difference
++ finite_affine_combination
++ zero

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) = (1.00, 0.00)
Current number Change Type
10169 1 backward.isDefEq

Current commit 412906b87e
Reference commit 151b5c4ae6

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 Mar 6, 2026
@grunweg grunweg changed the title feat: Covariant derivatives on vector bundles feat: covariant derivatives on vector bundles Mar 6, 2026
@grunweg grunweg added the t-differential-geometry Manifolds etc label Mar 6, 2026
/--
A covariant derivative ∇ is called of class `C^k` iff, whenever `X` is a `C^k` section and `σ` a
`C^{k+1}` section, the result `∇_X σ` is a `C^k` section. This is a class so typeclass inference can
deduce this automatically.
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.

Could you insist that there is no a priori monotonicity wrt k in this definition, if I understand correctly the definition? Or is there in fact such a monotonicity, although this is not obvious from the definition, thanks to a local description in coordinates?

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.

I added a comment. That stuff about differentiability is kind of a stub in this file, but we wanted to show we have in mind that the condition without differentiability assumption is a weird mathematical object. A later PR will develop the theory of covariant derivatives seen in a trivialization, including that you always get D + A where D is mfderic and A is an End(V)-valued 1-form.

@grunweg grunweg removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Mar 10, 2026
@mathlib-dependent-issues
Copy link
Copy Markdown

@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented Mar 10, 2026

@sgouezel Would you like to take another look at this PR, now that its dependency has been merged?
This would be very helpful - as Heather, Patrick and I were all involved with writing it, we better not merge it ourselves. Thanks!

@grunweg grunweg requested a review from sgouezel March 10, 2026 14:40
@ocfnash ocfnash added the awaiting-author A reviewer has asked the author a question or requested changes. label Mar 11, 2026
@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented Mar 12, 2026

Thanks for your comments. I have responded to them all; this PR is ready for review again.

Copy link
Copy Markdown
Contributor

@ocfnash ocfnash left a comment

Choose a reason for hiding this comment

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

Thanks, a few last comments but LGTM.

bors d+

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Mar 12, 2026

✌️ PatrickMassot can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@mathlib-triage mathlib-triage bot added delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Mar 12, 2026
@ocfnash
Copy link
Copy Markdown
Contributor

ocfnash commented Mar 12, 2026

Also

bors d=@grunweg

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Mar 12, 2026

✌️ grunweg can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@grunweg grunweg added the auto-merge-after-CI Please do not add manually. Requests for a bot to merge automatically once CI is done. label Mar 12, 2026
@mathlib-auto-merge
Copy link
Copy Markdown

As this PR is labelled auto-merge-after-CI, we are now sending it to bors:

bors merge

mathlib-bors bot pushed a commit that referenced this pull request Mar 12, 2026
From the path towards the the Levi-Civita connection and Riemannian geometry.

Co-authored-by: Heather Macbeth <25316162+hrmacbeth@users.noreply.github.com>
Co-authored-by: Michael Rothgang <rothgang@math.uni-bonn.de>
Co-authored-by: Heather Macbeth <25316162+hrmacbeth@users.noreply.github.com>
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Mar 12, 2026

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: covariant derivatives on vector bundles [Merged by Bors] - feat: covariant derivatives on vector bundles Mar 12, 2026
@mathlib-bors mathlib-bors bot closed this Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-after-CI Please do not add manually. Requests for a bot to merge automatically once CI is done. delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). t-differential-geometry Manifolds etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants