Skip to content

[Merged by Bors] - feat: associativity lemmas for MultilinearMap composition#25169

Closed
eric-wieser wants to merge 3 commits intomasterfrom
eric-wieser/multilinear-comp-lemmas
Closed

[Merged by Bors] - feat: associativity lemmas for MultilinearMap composition#25169
eric-wieser wants to merge 3 commits intomasterfrom
eric-wieser/multilinear-comp-lemmas

Conversation

@eric-wieser
Copy link
Copy Markdown
Member

Also tidies up the variables a bit to state these.


Open in Gitpod

@eric-wieser eric-wieser requested a review from urkud May 25, 2025 00:27
@eric-wieser eric-wieser added the t-algebra Algebra (groups, rings, fields, etc) label May 25, 2025
@github-actions
Copy link
Copy Markdown

PR summary 6256fe2d56

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ compMultilinearMap_compLinearMap
+ comp_compMultilinearMap
+ id_compMultilinearMap

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

@eric-wieser eric-wieser added the easy < 20s of review time. See the lifecycle page for guidelines. label May 25, 2025
Copy link
Copy Markdown
Contributor

@YaelDillies YaelDillies left a comment

Choose a reason for hiding this comment

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

Thanks!

maintainer merge

@github-actions
Copy link
Copy Markdown

🚀 Pull request has been placed on the maintainer queue by YaelDillies.

@github-actions github-actions bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label May 27, 2025
@eric-wieser eric-wieser requested a review from Vierkantor May 29, 2025 09:01
Copy link
Copy Markdown
Contributor

@j-loreaux j-loreaux left a comment

Choose a reason for hiding this comment

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

bors merge

@ghost ghost added ready-to-merge This PR has been sent to bors. and removed maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. labels May 29, 2025
mathlib-bors bot pushed a commit that referenced this pull request May 29, 2025
Also tidies up the variables a bit to state these.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented May 29, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: associativity lemmas for MultilinearMap composition [Merged by Bors] - feat: associativity lemmas for MultilinearMap composition May 29, 2025
@mathlib-bors mathlib-bors bot closed this May 29, 2025
@mathlib-bors mathlib-bors bot deleted the eric-wieser/multilinear-comp-lemmas branch May 29, 2025 14:16
bwehlin pushed a commit to bwehlin/mathlib4 that referenced this pull request May 31, 2025
joelriou pushed a commit that referenced this pull request Jun 8, 2025
Also tidies up the variables a bit to state these.
TOMILO87 pushed a commit that referenced this pull request Jun 8, 2025
Also tidies up the variables a bit to state these.
callesonne pushed a commit to callesonne/mathlib4 that referenced this pull request Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

easy < 20s of review time. See the lifecycle page for guidelines. ready-to-merge This PR has been sent to bors. t-algebra Algebra (groups, rings, fields, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants