-
Notifications
You must be signed in to change notification settings - Fork 21
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
Tradeoff of defining types as subobjects #86
base: master
Are you sure you want to change the base?
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,119 @@ | ||
--- | ||
author: 'Yaël Dillies' | ||
category: 'Papers' | ||
date: 2024-07-25 14:00:00 UTC+03:00 | ||
description: '' | ||
has_math: true | ||
link: '' | ||
slug: tradeoff-of-defining-types-as-subobjects | ||
tags: '' | ||
title: Tradeoffs of defining types as subobjects | ||
type: text | ||
--- | ||
|
||
It often happens in formalisation that a type of interest is a subobject of another type of interest. For example, the unit circle in the complex plane is naturally a submonoid (nevermind that it is actually a subgroup, we currently can't talk about subgroups of fields). | ||
YaelDillies marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
What is the best way of defining this unit circle in terms of `Complex`? This blog post examines the pros and cons of the available designs. | ||
|
||
## The available designs | ||
|
||
We will illustrate the various designs using the "circle in the complex plane" example. | ||
|
||
The first option, most loyal to the description of the unit circle as a submonoid of the complex plane, is to simply define the unit circle as a `Submonoid Complex`. We will call the "Subobject design". | ||
|
||
```lean | ||
-- Subobject design | ||
def circle : Submonoid Complex where | ||
carrier := {x : Complex | ‖x‖ = 1} | ||
one_mem' := sorry | ||
mul_mem' := sorry | ||
``` | ||
|
||
The second obvious choice would be to define the unit circle as a custom structure. We call this the "Custom Structure" design. | ||
|
||
```lean | ||
-- Custom Structure design | ||
structure Circle : Type where | ||
val : Complex | ||
norm_val : ‖val‖ = 1 | ||
``` | ||
|
||
Finally, an intermediate option would be to define it as the coercion to `Type` of a subobject. We call this the "Coerced Subobject" design. | ||
|
||
```lean | ||
-- Coerced Subobject design | ||
def Circle : Type := circle -- possibly replacing `circle` by its definition | ||
``` | ||
|
||
## Typeclass search | ||
|
||
In the Subobject design, all instances about `circle` are actually about `↥circle` where `↥` is [`CoeSort.coe`](https://leanprover-community.github.io/mathlib4_docs/find/?pattern=CoeSort.coe#doc). Eg `Foo circle` is secretly `Foo ↥circle`. This makes the head symbol be `CoeSort.coe`, meaning that typeclass search will try unifying *every* `Foo ↥someSubobject` instance with `Foo ↥circle`. In doing so, it will try unifying *every* subobject `someSubobject` with a `Foo` instance with `circle`. This is potentially really expensive. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is this still the case in Lean 4? I thought discrimination trees avoided this... There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Very much so. Else why would leanprover-community/mathlib4#12386 have achieved anything perf-wise? The problem I believe is that the discr tree key is |
||
|
||
The Coerced Subobject and Custom Structure designs do not suffer from this performance penalty. | ||
|
||
See [#12386](https://github.com/leanprover-community/mathlib4/pull/12386) for an example where switching from the Subobject design (`ringOfIntegers`) to the Coerced Subobject design (`RingOfIntegers`) dramatically increased performance. | ||
|
||
## Dot notation | ||
|
||
In the Subobject design, `x : circle` really means `x : ↥circle`, namely that `x` has type `Subtype _`. This means that dot notation `x.foo` resolves to `Subtype.foo x` rather than the expected `circle.foo x`. | ||
|
||
The Coerced Subobject and Custom Structure designs do not suffer from this (unexpected) behavior. | ||
|
||
See [#15021](https://github.com/leanprover-community/mathlib4/pull/15021) for an example where switching from the Subobject design (`ProbabilityTheory.kernel`) to the Custom Structure design (`ProbabilityTheory.Kernel`) was motivated by access to dot notation. | ||
|
||
## Custom named projections | ||
|
||
In the Subobject and Coerced Subobject designs, the fields of `↥circle`/`Circle` are `val` and `property`, as coming from `Subtype`. This leads to potentially uninforming code of the form | ||
```lean | ||
-- Subobject design | ||
def foo : circle where | ||
val := 1 | ||
property := by simp | ||
|
||
-- Coerced subobject design | ||
def bar : Circle where | ||
val := 1 | ||
property := by simp | ||
``` | ||
|
||
In the Custom Structure design, projections can be custom-named, leading to more informative code: | ||
```lean | ||
def baz : Circle where | ||
val := 1 | ||
norm_val := by simp | ||
``` | ||
|
||
## Generic instances | ||
|
||
In the Coerced Subobject and Custom Structure designs, generic instances about subobjects do not apply and must be copied over manually. In the Coerced Subobject design, they can easily be transferred/derived: | ||
```lean | ||
def Circle : Type := circle | ||
deriving TopologicalSpace | ||
|
||
instance : MetricSpace Circle := Subtype.metricSpace | ||
``` | ||
|
||
In the Custom Structure design, these instances must be either copied over manually or transferred some kind of isomorphism. | ||
YaelDillies marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
The Subobject design, by definition, lets all generic instances apply. | ||
|
||
## Conclusion | ||
|
||
Here is a table compiling the above discussion. | ||
|
||
| Aspect | Subobject | Coerced Subobject | Custom Structure | | ||
|--|--|--|--| | ||
| Typeclass search | Possibly expensive | Fast | Fast | | ||
| Dot notation | ✗ | ✓ | ✓ | | ||
| Custom named projections | ✗ | ✗ | ✓ | | ||
| Generic instances | automatic | easy to transfer | hard to transfer but could be improved | | ||
|
||
In conclusion, the Custom Structure design is superior in performance and usability at the expense of a one-time overhead cost when transferring generic instances. The Subobject design is still viable for types that see little use, so as to avoid the instance transferring cost. Finally, the Coerced Subobject is a good middle ground when having custom named projections is not particularly necessary. It could also reasonably be used as an intermediate step when refactoring from the Subobject design to the Custom Structure design. | ||
|
||
## Further concerns | ||
|
||
Subobjects are not the only ones having a coercion to `Type`: Concrete categories are categories whose objects can be interpreted as types, and there usually is a coercion from such a category to `Type`. | ||
|
||
For example, [`AlgebraicGeometry.Scheme`](https://leanprover-community.github.io/mathlib4_docs/find/?pattern=AlgebraicGeometry.Scheme#doc) sees widespread use in algebraic geometry and [`AlgebraicGeometry.Proj`](https://leanprover-community.github.io/mathlib4_docs/find/?pattern=AlgebraicGeometry.Proj#doc) is a term of type `Scheme` that is also used as a type. | ||
|
||
This raises issues of its own, as `Proj` is (mathematically) also a smooth manifold. Since terms only have one type, we can't hope to have both `Proj : Scheme` and `Proj : SmoothMnfld` (this last category does not exist yet in Mathlib). One option would be to have a separate `DifferentialGeometry.Proj` of type `SmoothMnfld`. Another one would be to provide the instances saying that `AlgebraicGeometry.Proj` is a smooth manifold. | ||
YaelDillies marked this conversation as resolved.
Show resolved
Hide resolved
YaelDillies marked this conversation as resolved.
Show resolved
Hide resolved
|
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.
Not sure what to put here