Skip to content
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] - fix(LinearAlgebra/Dimension): make LinearMap.rank an abbrev #6545

Closed
wants to merge 1 commit into from

Conversation

Julian
Copy link
Collaborator

@Julian Julian commented Aug 12, 2023

Previously, a statement like:

import Mathlib.LinearAlgebra.Dimension
import Mathlib.Tactic

variable {K : Type} [Field K]
variable {V W : Type} [AddCommGroup V] [AddCommGroup W] [Module K V] [Module K W]
variable {L : V →ₗ[K] W}

open LinearMap (ker)
open Module (rank)

example : rank K V = L.rank + rank K (ker L) := by
    sorry

required rewriting LinearMap.rank before manually finding the existence of rank_range_add_rank_ker.

After this change though, exact? successfully finds the lemma / solves the above.


Open in Gitpod

Previously, a statement like:

```
example : rank K V = L.rank + rank K (ker L) := by
    sorry
```

required rewriting `LinearMap.rank` before manually finding
the existence of `rank_range_add_rank_ker`.

After this change though, `exact?` successfully finds the lemma / solves
the above.
@Julian Julian added the awaiting-review The author would like community review of the PR label Aug 14, 2023
@ADedecker
Copy link
Member

Thanks!

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels Aug 18, 2023
bors bot pushed a commit that referenced this pull request Aug 18, 2023
Previously, a statement like:

```lean
import Mathlib.LinearAlgebra.Dimension
import Mathlib.Tactic

variable {K : Type} [Field K]
variable {V W : Type} [AddCommGroup V] [AddCommGroup W] [Module K V] [Module K W]
variable {L : V →ₗ[K] W}

open LinearMap (ker)
open Module (rank)

example : rank K V = L.rank + rank K (ker L) := by
    sorry
```

required rewriting `LinearMap.rank` before manually finding the existence of `rank_range_add_rank_ker`.

After this change though, `exact?` successfully finds the lemma / solves the above.
@bors
Copy link

bors bot commented Aug 18, 2023

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.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot changed the title fix(LinearAlgebra/Dimension): make LinearMap.rank an abbrev [Merged by Bors] - fix(LinearAlgebra/Dimension): make LinearMap.rank an abbrev Aug 18, 2023
@bors bors bot closed this Aug 18, 2023
@bors bors bot deleted the linearmap-rank-abbrev branch August 18, 2023 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has been sent to bors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants