-
Notifications
You must be signed in to change notification settings - Fork 234
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
[Merged by Bors] - feat: improve API for Fitting decomposition of a linear endomorphism #7487
Conversation
Especially the new lemma `LinearMap.eventually_isCompl_ker_pow_range_pow`
Can we add the words "Fitting decomposition" or Fitting lemma somewhere appropriate in the code? It seems odd that the only references to the naming would be in PR / commit messages and I can't see any reference to "fitting" in mathlib |
Good point; done in cd1ff11 |
@@ -267,15 +306,6 @@ theorem disjoint_partial_infs_eventually_top (f : ℕ → Submodule R M) | |||
exact (h m).eq_bot_of_ge (sup_eq_left.1 <| (w (m + 1) <| le_add_right p).symm.trans <| w m p) | |||
#align is_artinian.disjoint_partial_infs_eventually_top IsArtinian.disjoint_partial_infs_eventually_top | |||
|
|||
lemma _root_.LinearMap.exists_range_pow_eq_iInf (f : Module.End R M) : |
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.
This is removed because it has been replaced by the more powerful (and more useful) lemma: LinearMap.eventually_iInf_range_pow_eq
(above).
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.
This looks reasonable, but someone with more experience using filters than me should probably give a final look.
maintainer merge
🚀 Pull request has been placed on the maintainer queue by eric-wieser. |
bors d+ |
✌️ ocfnash can now approve this pull request. To approve and merge a pull request, simply reply with |
bors merge |
…7487) Especially the new lemma `LinearMap.eventually_isCompl_ker_pow_range_pow`
Pull request successfully merged into master. Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
Especially the new lemma
LinearMap.eventually_isCompl_ker_pow_range_pow