Conversation
|
This PR/issue depends on:
|
5be918f to
3089ff0
Compare
PR summary 3089ff0141Import changes for modified filesNo significant changes to the import graph Import changes for all files
|
| but don't use this assumption in the type. | ||
| (Instead, `(Pseudo)MetrisableSpace X` can suffice, or the assumption can be fully removed.) | ||
| -/ | ||
| @[env_linter] def metricMetrisable : Linter where |
There was a problem hiding this comment.
Note that we spell it as Metrizable in TopologicalSpace.MetrizableSpace
There was a problem hiding this comment.
My guess is that this linter doesn't catch anything, because normally you use data in [MetricSpace X], otherwise this would be an unused argument. The linter should catch theorems that only use the projection to UniformSpace. Another question is how to catch uses of [UniformSpace X] that only use projection to TopologicalSpace X (should be replaced by [TopologicalSpace X] [CompletelyRegularSpace X] once we redefine the latter to avoid dependency on Real) or [UniformSpace X] [Filter.IsCountablyGenerated (uniformity X)] that only use the projection to TopologicalSpace X everywhere else (should be replaced by [PseudoMetrizableSpace X].
|
Closing in favour of #35340. |
Similar to #10235, lint on occurrences of
MetricSpacewhich should beMetrisableSpace.Decidable,Fintype,Encodablelinters #10235 (for all the details)