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] - feat({Ring|Field}Theory/Adjoin): add various results on adjoin #9790

Closed
wants to merge 5 commits into from

Conversation

acmepjz
Copy link
Collaborator

@acmepjz acmepjz commented Jan 16, 2024

Main changes:

  • for intermediate fields: add extendScalars_adjoin, restrictScalars_adjoin_of_algEquiv, sup_toSubalgebra_of_isAlgebraic, adjoin_toSubalgebra_of_isAlgebraic, adjoin_rank_le_of_isAlgebraic, and golf sup_toSubalgebra
  • for subalgebras: add adjoin_eq_span_of_eq_span, adjoin_eq_span_basis, restrictScalars_adjoin, restrictScalars_adjoin_of_algEquiv, adjoin_rank_le

Open in Gitpod

@acmepjz acmepjz added awaiting-review The author would like community review of the PR awaiting-CI t-algebra Algebra (groups, rings, fields etc) labels Jan 16, 2024
Mathlib/FieldTheory/Adjoin.lean Outdated Show resolved Hide resolved
Mathlib/FieldTheory/Adjoin.lean Outdated Show resolved Hide resolved
Mathlib/FieldTheory/Adjoin.lean Show resolved Hide resolved
Mathlib/FieldTheory/Adjoin.lean Show resolved Hide resolved
@acmepjz acmepjz added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Jan 17, 2024
@acmepjz acmepjz added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Jan 17, 2024
Mathlib/FieldTheory/Adjoin.lean Outdated Show resolved Hide resolved
Mathlib/FieldTheory/Adjoin.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/Adjoin/Basic.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/Adjoin/Basic.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/Adjoin/Field.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/Adjoin/Basic.lean Outdated Show resolved Hide resolved
@riccardobrasca
Copy link
Member

There are still a couple of comments to be taken into account, but then LGTM, thanks!

bors d+

@mathlib-bors
Copy link

mathlib-bors bot commented Jan 19, 2024

✌️ acmepjz can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@github-actions github-actions bot added delegated and removed awaiting-review The author would like community review of the PR labels Jan 19, 2024
Co-authored-by: Junyan Xu <junyanxu.math@gmail.com>
@acmepjz
Copy link
Collaborator Author

acmepjz commented Jan 19, 2024

bors r+

mathlib-bors bot pushed a commit that referenced this pull request Jan 19, 2024
Main changes:

- for intermediate fields: add `extendScalars_adjoin`, `restrictScalars_adjoin_of_algEquiv`, `sup_toSubalgebra_of_isAlgebraic`, `adjoin_toSubalgebra_of_isAlgebraic`, `adjoin_rank_le_of_isAlgebraic`, and golf `sup_toSubalgebra`
- for subalgebras: add `adjoin_eq_span_of_eq_span`, `adjoin_eq_span_basis`, `restrictScalars_adjoin`, `restrictScalars_adjoin_of_algEquiv`, `adjoin_rank_le`
@mathlib-bors
Copy link

mathlib-bors bot commented Jan 19, 2024

Build failed:

@alreadydone alreadydone added the auto-merge-after-CI Please do not add manually. Requests for a bot to merge automatically once CI is done. label Jan 19, 2024
@leanprover-community-mathlib4-bot
Copy link
Collaborator

As this PR is labelled auto-merge-after-CI, we are now sending it to bors:

bors merge

mathlib-bors bot pushed a commit that referenced this pull request Jan 19, 2024
Main changes:

- for intermediate fields: add `extendScalars_adjoin`, `restrictScalars_adjoin_of_algEquiv`, `sup_toSubalgebra_of_isAlgebraic`, `adjoin_toSubalgebra_of_isAlgebraic`, `adjoin_rank_le_of_isAlgebraic`, and golf `sup_toSubalgebra`
- for subalgebras: add `adjoin_eq_span_of_eq_span`, `adjoin_eq_span_basis`, `restrictScalars_adjoin`, `restrictScalars_adjoin_of_algEquiv`, `adjoin_rank_le`



Co-authored-by: Junyan Xu <junyanxu.math@gmail.com>
@mathlib-bors
Copy link

mathlib-bors bot commented Jan 19, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat({Ring|Field}Theory/Adjoin): add various results on adjoin [Merged by Bors] - feat({Ring|Field}Theory/Adjoin): add various results on adjoin Jan 19, 2024
@mathlib-bors mathlib-bors bot closed this Jan 19, 2024
@mathlib-bors mathlib-bors bot deleted the acmepjz_adjoin_various branch January 19, 2024 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-after-CI Please do not add manually. Requests for a bot to merge automatically once CI is done. delegated t-algebra Algebra (groups, rings, fields etc)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants