mathlib icon indicating copy to clipboard operation
mathlib copied to clipboard

feat(algebra/module/submodule/lattice): `submodule.is_maximal`

Open haruhisa-enomoto opened this issue 2 years ago • 3 comments

This PR replaces the previous typeclass for maximal left ideals ideal.is_maximal : ideal R -> Prop with submodule.is_maximal : submodule R M -> Prop, which is a typeclass for maximal submodules of M. Some theorems are moved from namespace ideal to submodule so that for h : I.is_maximal for I : ideal R we can use dot notation h.is_prime.

Zulip


Open in Gitpod

haruhisa-enomoto avatar Jun 03 '22 00:06 haruhisa-enomoto

Could you please merge master? Otherwise LGTM. @eric-wieser, do you want to have another look? BTW, why most lemmas take (h : is_maximal _), not [is_maximal _]?

urkud avatar Jun 28 '22 12:06 urkud

:v: haruhisa-enomoto can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

bors[bot] avatar Jul 12 '22 13:07 bors[bot]

BTW, why most lemmas take (h : is_maximal _), not [is_maximal _]?

I believe this is only a class to allow R / I to have a field instance when is_maximal I. In other cases, because ideals have nontrivial equalities, working with classes is inconvenient.

Vierkantor avatar Jul 12 '22 13:07 Vierkantor