Skip to content

[Merged by Bors] - chore(Geometry/Manifold/VectorBundle/Basic): make some arguments implicit#26677

Closed
grunweg wants to merge 4 commits intoleanprover-community:masterfrom
grunweg:contMDiffAt-section-implicit
Closed

[Merged by Bors] - chore(Geometry/Manifold/VectorBundle/Basic): make some arguments implicit#26677
grunweg wants to merge 4 commits intoleanprover-community:masterfrom
grunweg:contMDiffAt-section-implicit

Conversation

@grunweg
Copy link
Copy Markdown
Contributor

@grunweg grunweg commented Jul 3, 2025

All of these are in iff lemmas, which are almost always used for rewriting, and occur on both sides of the equation.

Part of the path towards geodesics and the Levi-Civita connection.

Co-authored by: @PatrickMassot


Open in Gitpod

…pace}

These lemmas are usually used for rewriting, and s occurs on both the left
and right hand side.
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 3, 2025

PR summary 7672327e9c

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

No declarations were harmed in the making of this PR! 🐙

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

@github-actions github-actions bot added the t-differential-geometry Manifolds etc label Jul 3, 2025
@grunweg grunweg mentioned this pull request Jul 3, 2025
35 tasks
@grunweg grunweg added the easy < 20s of review time. See the lifecycle page for guidelines. label Jul 3, 2025
@Ruben-VandeVelde
Copy link
Copy Markdown
Contributor

Does the same argument not apply to contMDiffWithinAt_section?

@grunweg
Copy link
Copy Markdown
Contributor Author

grunweg commented Jul 3, 2025

Good point, it should. Let me try how this goes.

@grunweg grunweg added the awaiting-author A reviewer has asked the author a question or requested changes. label Jul 3, 2025
@grunweg grunweg changed the title chore: make section s implicit in contMDiff(Within)At_{section,totalSpace} chore(Geometry/Manifold/VectorBundle/Basic): make some arguments implicit Jul 3, 2025
@grunweg grunweg removed awaiting-author A reviewer has asked the author a question or requested changes. easy < 20s of review time. See the lifecycle page for guidelines. labels Jul 3, 2025
@grunweg
Copy link
Copy Markdown
Contributor Author

grunweg commented Jul 3, 2025

This seems to work well (it built locally, for instance): I'd say this is ready for review again.

@sgouezel
Copy link
Copy Markdown
Contributor

sgouezel commented Jul 3, 2025

Could you also do it for contMDiffWithinAt_totalSpace?
bors d+

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jul 3, 2025

✌️ 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.

@ghost ghost added the delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). label Jul 3, 2025
@grunweg
Copy link
Copy Markdown
Contributor Author

grunweg commented Jul 3, 2025

bors merge

mathlib-bors bot pushed a commit that referenced this pull request Jul 3, 2025
…icit (#26677)

All of these are in iff lemmas, which are almost always used for rewriting, and occur on both sides of the equation.

Part of the path towards geodesics and the Levi-Civita connection.

Co-authored by: @PatrickMassot
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jul 3, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore(Geometry/Manifold/VectorBundle/Basic): make some arguments implicit [Merged by Bors] - chore(Geometry/Manifold/VectorBundle/Basic): make some arguments implicit Jul 3, 2025
@mathlib-bors mathlib-bors bot closed this Jul 3, 2025
@grunweg grunweg deleted the contMDiffAt-section-implicit branch July 3, 2025 22:53
joelriou pushed a commit to joelriou/mathlib4 that referenced this pull request Jul 7, 2025
…icit (leanprover-community#26677)

All of these are in iff lemmas, which are almost always used for rewriting, and occur on both sides of the equation.

Part of the path towards geodesics and the Levi-Civita connection.

Co-authored by: @PatrickMassot
callesonne pushed a commit to callesonne/mathlib4 that referenced this pull request Jul 24, 2025
…icit (leanprover-community#26677)

All of these are in iff lemmas, which are almost always used for rewriting, and occur on both sides of the equation.

Part of the path towards geodesics and the Levi-Civita connection.

Co-authored by: @PatrickMassot
hrmacbeth pushed a commit to szqzs/mathlib4 that referenced this pull request Jul 28, 2025
…icit (leanprover-community#26677)

All of these are in iff lemmas, which are almost always used for rewriting, and occur on both sides of the equation.

Part of the path towards geodesics and the Levi-Civita connection.

Co-authored by: @PatrickMassot
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

3 participants