Commit 2024-06-11 07:52 fa87862b
View on Github →chore: fix diamond for WithTop subtraction (#13715)
We have currently two different subtractions on WithTop α
.
- The first one, requiring that
α
has a zero, mapsx - ⊤
to0
, and is the right convention when defining subtraction on ENat or ENNReal. - The second one, requiring a
LinearOrderedAddCommGroup
, mapsx - ⊤
to⊤
. It is the additivization of the usual conventionx / 0 = 0
, and is relevant to work with valuations. Both instances apply for example toWithTop ℤ
, creating a diamond. This PR changes the first instance to require a bottom element instead of a zero, to make sure that the two instances will apply in disjoint situations. This requires some new API for the second instance, to fixMathlib/Analysis/Analytic/Meromorphic.lean
which was (incorrectly) using the first instance.