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] - chore(linear_algebra/dimension): more same-universe versions of is_basis.mk_eq_dim #4591

Closed
wants to merge 1 commit into from

Conversation

urkud
Copy link
Member

@urkud urkud commented Oct 13, 2020

While all the lift magic is good for general theory, it is not that convenient for the case when everything is in Type.

  • add mk_eq_mk_of_basis': same-universe version of mk_eq_mk_of_basis;
  • generalize is_basis.mk_eq_dim'' to any index type in the same universe as V, not necessarily s : set V;
  • reorder lemmas to optimize the total length of the proofs;
  • drop one finite_dimensional assumption in findim_of_infinite_dimensional.

@urkud urkud added the awaiting-review The author would like community review of the PR label Oct 13, 2020
@fpvandoorn
Copy link
Member

LGTM

bors merge

@github-actions github-actions bot added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels Oct 13, 2020
bors bot pushed a commit that referenced this pull request Oct 13, 2020
…asis.mk_eq_dim` (#4591)

While all the `lift` magic is good for general theory, it is not that convenient for the case when everything is in `Type`.

* add `mk_eq_mk_of_basis'`: same-universe version of `mk_eq_mk_of_basis`;
* generalize `is_basis.mk_eq_dim''` to any index type in the same universe as `V`, not necessarily `s : set V`;
* reorder lemmas to optimize the total length of the proofs;
* drop one `finite_dimensional` assumption in `findim_of_infinite_dimensional`.
@bors
Copy link

bors bot commented Oct 13, 2020

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore(linear_algebra/dimension): more same-universe versions of is_basis.mk_eq_dim [Merged by Bors] - chore(linear_algebra/dimension): more same-universe versions of is_basis.mk_eq_dim Oct 13, 2020
@bors bors bot closed this Oct 13, 2020
@bors bors bot deleted the dim-lemmas branch October 13, 2020 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants