Skip to content

chore: remove shortcut instance that is no longer needed#38284

Open
jcommelin wants to merge 1 commit intoleanprover-community:masterfrom
jcommelin:techdebt-maxhb
Open

chore: remove shortcut instance that is no longer needed#38284
jcommelin wants to merge 1 commit intoleanprover-community:masterfrom
jcommelin:techdebt-maxhb

Conversation

@jcommelin
Copy link
Copy Markdown
Member


Open in Gitpod

@jcommelin
Copy link
Copy Markdown
Member Author

!bench

@leanprover-radar
Copy link
Copy Markdown

leanprover-radar commented Apr 20, 2026

Benchmark results for 70ff133 against 896cc56 are in. No significant results found. @jcommelin

  • build//instructions: -16.7G (-0.01%)

Small changes (1🟥)

  • 🟥 build/module/Mathlib.Tactic.MinImports//instructions: +132.8M (+2.64%)

@github-actions
Copy link
Copy Markdown

PR summary 896cc56a39

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

- instance {p : ℝ≥0∞} [Fact (1 ≤ p)] :

You can run this locally as follows
## summary with just the declaration names:
./scripts/pr_summary/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/pr_summary/declarations_diff.sh long <optional_commit>

The doc-module for scripts/pr_summary/declarations_diff.sh contains some details about this script.


No changes to technical debt.

You can run this locally as

./scripts/reporting/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-analysis Analysis (normed *, calculus) label Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-analysis Analysis (normed *, calculus)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants