Commit 2024-10-26 13:22 a9b268f6

View on Github →

refactor: change the definition of 1 : Submodule R A to the range of (· • 1) (#18092) Previously, it's defined to be the range of algebraMap R A. In the noncommutative setting, we want to write 1 : Ideal R where Ideal R := Submodule R R, but R isn't an R-algebra if R is not commutative. If we were to introduce a new typeclass by removing the commutes' field from Algebra, we could keep the current definition, but I'd argue that it's still better to use SMul to define 1 : Submodule R A, because Submodule only depends on the Module/SMul, not on the algebraMap/RingHom. Requires fixes in 11 other files.

Estimated changes