Skip to content

Commit bd7efb4

Browse files
committed
doc: clarify the scope of field_simp (leanprover-community#39869)
The term semi-field is sufficiently non-standard that it has led to confusion whether e.g. `NNReal` is supported by field_simp: it is (as it is a semifield). Let's clarify the doc-string. Inspired by discussions at the ICERM workshop "Techniques and Tools for the Formalization of Analysis".
1 parent da26519 commit bd7efb4

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

Mathlib/Tactic/FieldSimp.lean

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -665,10 +665,10 @@ def parseDischarger (d : Option (TSyntax ``discharger)) (args : Option (TSyntax
665665
| _ => throwError "could not parse the provided discharger {d}"
666666

667667
/--
668-
`field_simp` normalizes expressions in (semi-)fields by rewriting them to a common denominator,
669-
i.e. to reduce them to expressions of the form `n / d` where neither `n` nor `d` contains any
670-
division symbol. The `field_simp` tactic will also clear denominators in field *(in)equalities*, by
671-
cross-multiplying.
668+
`field_simp` normalizes expressions in (semi-)fields (i.e., does not require additive inverses)
669+
by rewriting them to a common denominator, i.e. to reduce them to expressions of the form `n / d`
670+
where neither `n` nor `d` contains any division symbol. The `field_simp` tactic will also clear
671+
denominators in field *(in)equalities*, by cross-multiplying.
672672
673673
A very common pattern is `field_simp; ring` (clear denominators, then the resulting goal is
674674
solvable by the axioms of a commutative ring). The finishing tactic `field` is a shorthand for this

0 commit comments

Comments
 (0)