Commit 2024-10-03 19:18 0c13e183

View on Github →

chore: Rename FiniteDimensional.finrank to Module.finrank (#17192) The namespacing in the linear algebra/affine spaces part of the library is a bit suboptimal. Here are a few arguments for why FiniteDimensional.finrank should be renamed to Module.finrank:

  1. Other objects in maths can be finite-dimensional and have a rank.
  2. finrank is directly analogous to Module.rank, they have lemmas in common (eg finrank_eq_rank, which is currently in neither of the declarations' namespaces) and have common series of lemmas that relate to other declarations in the Module namespace, eg Module.Finite.
  3. We are in the process of replacing FiniteDimensional by Module.Finite, so the FiniteDimensional namespace will soon be obsolete.
  4. It is a shorter and clearer name. Also make Module.Finite protected to avoid clashes with Finite

Estimated changes