Skip to content

feat: Gram-Schmidt orthonormalisation preserves continuity of sections#27694

Open
grunweg wants to merge 23 commits into
leanprover-community:masterfrom
grunweg:gramSchmidt-sections-cont
Open

feat: Gram-Schmidt orthonormalisation preserves continuity of sections#27694
grunweg wants to merge 23 commits into
leanprover-community:masterfrom
grunweg:gramSchmidt-sections-cont

Conversation

@grunweg

@grunweg grunweg commented Jul 30, 2025

Copy link
Copy Markdown
Contributor

provided the bundle metric used in the process is continuous.

From the path towards geodesics and the Levi-Civita connection.
Co-authored-by: Patrick Massot patrickmassot@free.fr


Baby version of #27025.

Open in Gitpod

@grunweg grunweg added the t-differential-geometry Manifolds etc label Jul 30, 2025
@github-actions

Copy link
Copy Markdown

PR summary 0ccc06d21e

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference
Mathlib.Geometry.Manifold.VectorBundle.GramSchmidtOrtho (new file) 2097

Declarations diff

+ ContinuousWithinAt.starProjection
+ coe_gramSchmidtBasis
+ coe_gramSchmidtNormedBasis
+ continuousAt_aux
+ continuousWithinAt_aux
+ continuousWithinAt_inner
+ gramSchmidt
+ gramSchmidtBasis
+ gramSchmidtNormed
+ gramSchmidtNormedBasis
+ gramSchmidtNormed_apply_of_orthogonal
+ gramSchmidtNormed_apply_of_orthonormal
+ gramSchmidtNormed_coe
+ gramSchmidtNormed_continuous
+ gramSchmidtNormed_continuousAt
+ gramSchmidtNormed_continuousOn
+ gramSchmidtNormed_continuousWithinAt
+ gramSchmidtNormed_linearIndependent
+ gramSchmidtNormed_orthonormal
+ gramSchmidtNormed_orthonormal'
+ gramSchmidtNormed_unit_length'
+ gramSchmidtOrthonormalBasis
+ gramSchmidtOrthonormalBasis_apply_of_orthonormal
+ gramSchmidtOrthonormalBasis_coe
+ gramSchmidt_apply
+ gramSchmidt_bot
+ gramSchmidt_continuous
+ gramSchmidt_continuousAt
+ gramSchmidt_continuousOn
+ gramSchmidt_continuousWithinAt
+ gramSchmidt_def
+ gramSchmidt_def'
+ gramSchmidt_def''
+ gramSchmidt_inv_triangular
+ gramSchmidt_linearIndependent
+ gramSchmidt_mem_span
+ gramSchmidt_ne_zero
+ gramSchmidt_ne_zero_coe
+ gramSchmidt_orthogonal
+ gramSchmidt_pairwise_orthogonal
+ gramSchmidt_zero
+ mem_span_gramSchmidt
+ span_gramSchmidt
+ span_gramSchmidtNormed
+ span_gramSchmidtNormed_range
+ span_gramSchmidt_Iic
+ span_gramSchmidt_Iio
+-+ gramSchmidt_of_orthogonal
-++ gramSchmidtNormed_unit_length
-++ gramSchmidtNormed_unit_length_coe

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

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

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


No changes to technical debt.

You can run this locally as

./scripts/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).

@mathlib4-dependent-issues-bot

mathlib4-dependent-issues-bot commented Jul 30, 2025

Copy link
Copy Markdown
Collaborator

@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Jul 30, 2025
@mathlib4-merge-conflict-bot mathlib4-merge-conflict-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 Nov 16, 2025
@mathlib4-merge-conflict-bot

Copy link
Copy Markdown
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) t-differential-geometry Manifolds etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants