chore(Algebra/Order): deduplicate material on ordered algebras
PR summary 0a5fe12b77
Import changes for modified files
Dependency changes
| File | Base Count | Head Count | Change |
|---|---|---|---|
| Mathlib.Algebra.Order.Algebra | 299 | 300 | +1 (+0.33%) |
| Mathlib.Algebra.Order.Module.Algebra | 300 | 301 | +1 (+0.33%) |
Import changes for all files
| Files | Import difference |
|---|---|
Mathlib.Algebra.Order.Algebra Mathlib.Algebra.Order.Module.Algebra |
1 |
Declarations diff
No declarations were harmed in the making of this PR! 🐙
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
relativevalue is the weighted sum of the differences with weight given by the inverse of the current value of the statistic. - The
absolutevalue is therelativevalue divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).
In my real closed field repo, I had defined ordered algebras because I couldn't find them in Mathlib. Today I found out they have been "defined" in Mathlib not once but twice, 3 years apart and apparently completely independently. How can we improve discoverability of this slightly counter-intuitive formalisation?
Moving it to
Algebra.Order.Algebrawould help
OK, I will switch the module deprecation.
🚀 Pull request has been placed on the maintainer queue by YaelDillies.
Thanks :tada:
bors merge
@jcommelin fixed
maintainer merge
🚀 Pull request has been placed on the maintainer queue by YaelDillies.
Spurious CI failure
Thanks :tada:
bors merge
Hijacking this PR to run a quick test for #33042. Please ignore! maintainer merge?
🚀 Pull request has been placed on the maintainer queue by bryangingechen.