Skip to content

[Merged by Bors] - feat: add module instance to ContDiffMapSupportedIn#30198

Closed
luigi-massacci wants to merge 34 commits intoleanprover-community:masterfrom
luigi-massacci:LM_PR_ContDiff_module
Closed

[Merged by Bors] - feat: add module instance to ContDiffMapSupportedIn#30198
luigi-massacci wants to merge 34 commits intoleanprover-community:masterfrom
luigi-massacci:LM_PR_ContDiff_module

Conversation

@luigi-massacci
Copy link
Copy Markdown
Collaborator

@luigi-massacci luigi-massacci commented Oct 4, 2025

Add a module instance for the type ContDiffMapSupportedIn of n-times continuously differentiable maps supported in a fixed compact.


Open in Gitpod

@github-actions github-actions bot added the t-analysis Analysis (normed *, calculus) label Oct 4, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 4, 2025

PR summary 45dfe1bcb6

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.Analysis.Distribution.ContDiffMapSupportedIn 1761 1779 +18 (+1.02%)
Import changes for all files
Files Import difference
Mathlib.Analysis.Distribution.ContDiffMapSupportedIn 18

Declarations diff

+ EqOn.comp_left₂
+ add_apply
+ coeHom
+ coeHom_injective
+ coe_add
+ coe_coeHom
+ coe_neg
+ coe_smul
+ coe_sub
+ coe_zero
+ instSMul
+ instSub
+ instance : Add 𝓓^{n}_{K}(E, F)
+ instance : AddCommGroup 𝓓^{n}_{K}(E, F)
+ instance : Neg 𝓓^{n}_{K}(E, F)
+ instance : Zero 𝓓^{n}_{K}(E, F)
+ instance {R} [Semiring R] [Module R F] [SMulCommClass ℝ R F] [ContinuousConstSMul R F] :
+ neg_apply
+ smul_apply
+ sub_apply
+ toBoundedContinuousFunction_apply
+ zero_apply

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

@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented Oct 17, 2025

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 Oct 17, 2025
mathlib-bors bot pushed a commit that referenced this pull request Oct 17, 2025
Add a module instance for the type `ContDiffMapSupportedIn` of n-times continuously differentiable maps supported in a fixed compact.

Co-authored-by: Michael Rothgang <rothgang@math.uni-bonn.de>
Co-authored-by: Anatole Dedecker <anatolededecker@gmail.com>
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Oct 17, 2025

Build failed (retrying...):

@bryangingechen
Copy link
Copy Markdown
Contributor

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Oct 17, 2025

✌️ luigi-massacci 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-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Oct 17, 2025

Canceled.

@ghost ghost added delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). and removed ready-to-merge This PR has been sent to bors. delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). labels Oct 17, 2025
@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented Oct 17, 2025

Sorry for the failed batch; let's try again.
bors merge

@ghost ghost added the ready-to-merge This PR has been sent to bors. label Oct 17, 2025
mathlib-bors bot pushed a commit that referenced this pull request Oct 17, 2025
Add a module instance for the type `ContDiffMapSupportedIn` of n-times continuously differentiable maps supported in a fixed compact.

Co-authored-by: Michael Rothgang <rothgang@math.uni-bonn.de>
Co-authored-by: Anatole Dedecker <anatolededecker@gmail.com>
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Oct 17, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: add module instance to ContDiffMapSupportedIn [Merged by Bors] - feat: add module instance to ContDiffMapSupportedIn Oct 17, 2025
@mathlib-bors mathlib-bors bot closed this Oct 17, 2025
Jlh18 pushed a commit to Jlh18/mathlib4 that referenced this pull request Oct 24, 2025
…nity#30198)

Add a module instance for the type `ContDiffMapSupportedIn` of n-times continuously differentiable maps supported in a fixed compact.

Co-authored-by: Michael Rothgang <rothgang@math.uni-bonn.de>
Co-authored-by: Anatole Dedecker <anatolededecker@gmail.com>
grunweg added a commit to grunweg/mathlib4 that referenced this pull request Oct 30, 2025
…mmas about instances

PR leanprover-community#30198 added both a coe_foo and a foo_apply lemma for foo in
{zero,add,sub,neg,smul}: remove the apply lemmas again.
- there was no good reason for having both; when in doubt, use coe lemmas
- for simp, the coe lemmas are equally strong (and having both is very unusual
  at best),
- when rewriting, one can add Pi.zero_apply and friends.
grunweg added a commit to grunweg/mathlib4 that referenced this pull request Oct 30, 2025
…mmas about instances

PR leanprover-community#30198 added both a coe_foo and a foo_apply lemma for foo in
{zero,add,sub,neg,smul}: remove the apply lemmas again.
- there was no good reason for having both; when in doubt, use coe lemmas
- for simp, the coe lemmas are equally strong (and having both is very unusual
  at best),
- when rewriting, one can add Pi.zero_apply and friends.
mathlib-bors bot pushed a commit that referenced this pull request Oct 30, 2025
#31094)

…mmas about instances

#30198 added both a coe_foo and a foo_apply lemma for foo in {zero,add,sub,neg,smul}: remove the apply lemmas again.
- there was no good reason for having both; when in doubt, use coe lemmas
- for simp, the coe lemmas are equally strong (and having both is very unusual at best),
- when rewriting, one can add Pi.zero_apply and friends.
mathlib-bors bot pushed a commit that referenced this pull request Oct 31, 2025
BeibeiX0 pushed a commit to BeibeiX0/mathlib4 that referenced this pull request Nov 7, 2025
…nity#30198)

Add a module instance for the type `ContDiffMapSupportedIn` of n-times continuously differentiable maps supported in a fixed compact.

Co-authored-by: Michael Rothgang <rothgang@math.uni-bonn.de>
Co-authored-by: Anatole Dedecker <anatolededecker@gmail.com>
BeibeiX0 pushed a commit to BeibeiX0/mathlib4 that referenced this pull request Nov 7, 2025
leanprover-community#31094)

…mmas about instances

leanprover-community#30198 added both a coe_foo and a foo_apply lemma for foo in {zero,add,sub,neg,smul}: remove the apply lemmas again.
- there was no good reason for having both; when in doubt, use coe lemmas
- for simp, the coe lemmas are equally strong (and having both is very unusual at best),
- when rewriting, one can add Pi.zero_apply and friends.
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-analysis Analysis (normed *, calculus)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants