Skip to content

[Merged by Bors] - feat: lemmas about Finset.disjSum#23078

Closed
eric-wieser wants to merge 1 commit intomasterfrom
eric-wieser/disjSum-fold
Closed

[Merged by Bors] - feat: lemmas about Finset.disjSum#23078
eric-wieser wants to merge 1 commit intomasterfrom
eric-wieser/disjSum-fold

Conversation

@eric-wieser
Copy link
Copy Markdown
Member

Split from #22983, since these don't need any new imports.


Open in Gitpod

@github-actions
Copy link
Copy Markdown

PR summary 6d1d8fb6ae

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ fold_disjSum
++ map_disjSum

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-data Data (lists, quotients, numbers, etc) label Mar 18, 2025
theorem map_disjSum (f : α ⊕ β ↪ γ) :
(s.disjSum t).map f =
(s.map (.trans .inl f)).disjUnion (t.map (.trans .inr f)) (by
as_aux_lemma =>
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.

woah, what's this

@b-mehta
Copy link
Copy Markdown
Contributor

b-mehta commented Mar 19, 2025

Nice!

bors merge

@ghost ghost added the ready-to-merge This PR has been sent to bors. label Mar 19, 2025
mathlib-bors bot pushed a commit that referenced this pull request Mar 19, 2025
Split from #22983, since these don't need any new imports.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Mar 19, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: lemmas about Finset.disjSum [Merged by Bors] - feat: lemmas about Finset.disjSum Mar 19, 2025
@mathlib-bors mathlib-bors bot closed this Mar 19, 2025
@mathlib-bors mathlib-bors bot deleted the eric-wieser/disjSum-fold branch March 19, 2025 01:40
tukamilano pushed a commit that referenced this pull request Mar 20, 2025
Split from #22983, since these don't need any new imports.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. t-data Data (lists, quotients, numbers, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants