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.leanwhich was (incorrectly) using the first instance.