Skip to content

Conversation

@yuanyi-350
Copy link
Collaborator


Open in Gitpod

@github-actions github-actions bot added large-import Automatically added label for PRs with a significant increase in transitive imports t-analysis Analysis (normed *, calculus) labels Jan 17, 2026
@github-actions
Copy link

github-actions bot commented Jan 17, 2026

PR summary 20b873c44e

Import changes exceeding 2%

% File
+11.49% Mathlib.Analysis.SpecialFunctions.Trigonometric.Chebyshev.RootsExtrema

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.Analysis.SpecialFunctions.Trigonometric.Chebyshev.RootsExtrema 1924 2145 +221 (+11.49%)
Mathlib.NumberTheory.Niven 1945 1948 +3 (+0.15%)
Import changes for all files
Files Import difference
Mathlib.NumberTheory.Niven 3
Mathlib.Analysis.SpecialFunctions.Trigonometric.Chebyshev.RootsExtrema 221

Declarations diff

+ irrational_cos_chebyshev_angle
+ irrational_of_root_T_real

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

@YuvalFilmus
Copy link
Collaborator

I have a competing PR #34044.
Not enough of an expert to say which one is better...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

large-import Automatically added label for PRs with a significant increase in transitive imports t-analysis Analysis (normed *, calculus)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants