-
Notifications
You must be signed in to change notification settings - Fork 839
chore: deprecate monoidalOfHasFiniteProducts
#30684
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
chore: deprecate monoidalOfHasFiniteProducts
#30684
Conversation
Over the summer, this was replaced everywhere with `CartesianMonoidalCategory.ofHasFiniteProducts`, but hadn't been deprecated.
PR summary 7c64d37d34Import changes exceeding 2%
|
File | Base Count | Head Count | Change |
---|---|---|---|
Mathlib.CategoryTheory.Monoidal.OfHasFiniteProducts | 510 | 653 | +143 (+28.04%) |
Import changes for all files
Files | Import difference |
---|---|
Mathlib.CategoryTheory.Monoidal.OfHasFiniteProducts |
143 |
Declarations diff
+ instance (X Y : C) :
+++ instance :
- instance (X Y : C) : IsIso (δ F X Y) := by dsimp [δ_eq]; infer_instance
- instance : F.Monoidal := .ofOplaxMonoidal F
- instance : F.OplaxMonoidal
- instance : IsIso (η F) := by dsimp [η_eq]; infer_instance
- leftUnitor_inv
- rightUnitor_inv
- tensorHom
- whiskerLeft
- whiskerRight
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.
Increase in tech debt: (relative, absolute) = (0.71, 0.01)
Current number | Change | Type |
---|---|---|
753 | -3 | erw |
59 | 1 | disabled deprecation lints |
Current commit e4f59d6725
Reference commit 7c64d37d34
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 therelative
value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).
section | ||
|
||
/-- A category with a terminal object and binary products has a natural monoidal structure. -/ | ||
@[deprecated CartesianMonoidalCategory.ofHasFiniteProducts (since := "2025-10-19")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the definition itself, could you use CartesianMonoidalCategory.ofHasFiniteProducts
? (using hasFiniteProductsOfHasBinaryAndTerminal
).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, although I had to drop the lemmas that are not true anymore
Over the summer, this was replaced everywhere with
CartesianMonoidalCategory.ofHasFiniteProducts
, but hadn't been deprecated.I would personally be very happy to also delete the finite coproduct stuff, but we don't yet have cocartesian-monoidal categories to replace them.