Skip to content

[Merged by Bors] - feat(Analysis/SpecialFunctions): iterated derivative of cotangent#27212

Closed
CBirkbeck wants to merge 58 commits intoleanprover-community:masterfrom
CBirkbeck:cot_series_iteratedDerivWithin
Closed

[Merged by Bors] - feat(Analysis/SpecialFunctions): iterated derivative of cotangent#27212
CBirkbeck wants to merge 58 commits intoleanprover-community:masterfrom
CBirkbeck:cot_series_iteratedDerivWithin

Conversation

@CBirkbeck
Copy link
Copy Markdown
Collaborator

@CBirkbeck CBirkbeck commented Jul 16, 2025

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 16, 2025

PR summary 0ad288bf4a

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.Analysis.SpecialFunctions.Trigonometric.Cotangent 2496 2498 +2 (+0.08%)
Import changes for all files
Files Import difference
Mathlib.Analysis.SpecialFunctions.Trigonometric.Cotangent 2

Declarations diff

+ abs_norm_eq_max_natAbs
+ abs_norm_eq_max_natAbs_neg
+ cot_pi_mul_contDiffWithinAt
+ differentiableOn_iteratedDerivWithin_cotTerm
+ eqOn_iteratedDerivWithin_cotTerm_integerComplement
+ eqOn_iteratedDerivWithin_cotTerm_upperHalfPlaneSet
+ eqOn_iteratedDeriv_cotTerm
+ iteratedDerivWithin_cot_pi_mul_eq_mul_tsum_div_pow
+ iteratedDerivWithin_cot_pi_mul_eq_mul_tsum_zpow
+ iteratedDerivWithin_cot_sub_inv_eq_add_mul_tsum
+ sin_pi_mul_ne_zero
+ summableLocallyUniformlyOn_iteratedDerivWithin_cotTerm
+ upperHalfPlane_inter_integerComplement
- sin_pi_z_ne_zero

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 added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Jul 16, 2025
CBirkbeck and others added 3 commits September 5, 2025 09:10
Co-authored-by: Michael Stoll <99838730+MichaelStollBayreuth@users.noreply.github.com>
@CBirkbeck
Copy link
Copy Markdown
Collaborator Author

Can you try to be more consistent regarding lemma vs. theorem? I think the idea is that theorems are the more interesting statments (and should have docstrings), whereas lemma is for the auxiliary and less interesting or boring stuff. (mathport turned all lemmas into theorems, so this is not very consistent in current Mathlib, but I think it makes sense to do that for new stuff.)

Sorry I was still under the impression that lemma and theorem were treated the same in mathlib. These were named theorems either from the mathport, but also extract_goal automatically calls them theorems and I never changed that bit. But I think it would be nice to have it as you say so I'll do that going forward thanks!

Comment thread Mathlib/Analysis/SpecialFunctions/Trigonometric/Cotangent.lean Outdated
@MichaelStollBayreuth MichaelStollBayreuth added the awaiting-author A reviewer has asked the author a question or requested changes. label Sep 5, 2025
@CBirkbeck CBirkbeck removed the awaiting-author A reviewer has asked the author a question or requested changes. label Sep 5, 2025
Comment thread Mathlib/Analysis/SpecialFunctions/Trigonometric/Cotangent.lean Outdated
Comment thread Mathlib/Analysis/SpecialFunctions/Trigonometric/Cotangent.lean Outdated
@MichaelStollBayreuth
Copy link
Copy Markdown
Contributor

Instead of fun x ↦ π * Complex.cot (π * x) you could write fun x : ℂ ↦ π * cot (π * x) (the Complex namespace is open, and specifying the type of x resolves the ambiguity), but this is a matter of taste.

@MichaelStollBayreuth MichaelStollBayreuth added the awaiting-author A reviewer has asked the author a question or requested changes. label Sep 5, 2025
CBirkbeck and others added 2 commits September 8, 2025 14:19
Co-authored-by: Michael Stoll <99838730+MichaelStollBayreuth@users.noreply.github.com>
@CBirkbeck CBirkbeck removed the awaiting-author A reviewer has asked the author a question or requested changes. label Sep 8, 2025
@MichaelStollBayreuth
Copy link
Copy Markdown
Contributor

LGTM now.

maintainer merge

@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 8, 2025

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

@ghost ghost added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Sep 8, 2025
@riccardobrasca
Copy link
Copy Markdown
Member

Thanks!

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 Sep 8, 2025
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Sep 8, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Analysis/SpecialFunctions): iterated derivative of cotangent [Merged by Bors] - feat(Analysis/SpecialFunctions): iterated derivative of cotangent Sep 8, 2025
@mathlib-bors mathlib-bors bot closed this Sep 8, 2025
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-number-theory Number theory (also use t-algebra or t-analysis to specialize)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants