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(geometry/manifold/vector_bundle/basic): smooth vector bundles #17611

Closed
wants to merge 50 commits into from

Conversation

hrmacbeth
Copy link
Member

@hrmacbeth hrmacbeth commented Nov 18, 2022

Definition of smooth vector bundle, and basic constructions (direct sum, pullback, core construction).

Co-authored-by: Floris van Doorn fpvdoorn@gmail.com


Open in Gitpod

@fpvandoorn fpvandoorn added the awaiting-CI The author would like to see what CI has to say before doing more work. label Feb 27, 2023
@fpvandoorn
Copy link
Member

merged with master. I'll make changes to incorporate my comment, but feel free to already review

@fpvandoorn fpvandoorn removed the awaiting-author A reviewer has asked the author a question or requested changes label Feb 28, 2023
@github-actions github-actions bot removed the awaiting-CI The author would like to see what CI has to say before doing more work. label Feb 28, 2023
Copy link
Collaborator

@sgouezel sgouezel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to be working very nicely, congrats!
bors d+

src/geometry/manifold/vector_bundle/fiberwise_linear.lean Outdated Show resolved Hide resolved
src/geometry/manifold/vector_bundle/basic.lean Outdated Show resolved Hide resolved

## Main definitions and constructions

* `fiber_bundle.charted_space`: A fibre bundle `E` over a base `B` with model fibre `F` is naturally
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks strange to me that the Lean name has fiber_bundle but the docstring talks of fibre bundle. I would uniformize that (or uniformise that, as you like), maybe in a later PR.

src/geometry/manifold/vector_bundle/basic.lean Outdated Show resolved Hide resolved
src/geometry/manifold/vector_bundle/basic.lean Outdated Show resolved Hide resolved
src/geometry/manifold/vector_bundle/basic.lean Outdated Show resolved Hide resolved
src/geometry/manifold/vector_bundle/basic.lean Outdated Show resolved Hide resolved
src/geometry/manifold/vector_bundle/pullback.lean Outdated Show resolved Hide resolved
@bors
Copy link

bors bot commented Mar 3, 2023

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

@leanprover-community-bot-assistant leanprover-community-bot-assistant added delegated The PR author may merge after reviewing final suggestions. and removed awaiting-review The author would like community review of the PR labels Mar 3, 2023
@fpvandoorn
Copy link
Member

bors merge

@github-actions github-actions bot added the ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) label Mar 7, 2023
bors bot pushed a commit that referenced this pull request Mar 7, 2023
…17611)

Definition of smooth vector bundle, and basic constructions (direct sum, pullback, core construction).

Co-authored-by: Floris van Doorn <fpvdoorn@gmail.com>



Co-authored-by: Floris van Doorn <fpvdoorn@gmail.com>
@bors
Copy link

bors bot commented Mar 7, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(geometry/manifold/vector_bundle/basic): smooth vector bundles [Merged by Bors] - feat(geometry/manifold/vector_bundle/basic): smooth vector bundles Mar 7, 2023
@bors bors bot closed this Mar 7, 2023
@bors bors bot deleted the smooth-vector-bundle-pr branch March 7, 2023 20:20
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 11, 2023
SHA-only updates:
* leanprover-community/mathlib#17611 – `CategoryTheory.Sites.Sheaf` already uses `aesop_cat` at this location, nothing to port.
* leanprover-community/mathlib#18742 – fiber spelling, which is all of the changes to `Topology.FiberBundle.Trivialization`, is already in mathlib4.
* leanprover-community/mathlib#18198 – `FunLike` change to `Data.PEquiv` is already in mathlib4.

Substantative forward port:
* leanprover-community/mathlib#18520



Co-authored-by: Parcly Taxel <reddeloostw@gmail.com>
eric-wieser pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 12, 2023
SHA-only updates:
* leanprover-community/mathlib#17611 – `CategoryTheory.Sites.Sheaf` already uses `aesop_cat` at this location, nothing to port.
* leanprover-community/mathlib#18742 – fiber spelling, which is all of the changes to `Topology.FiberBundle.Trivialization`, is already in mathlib4.
* leanprover-community/mathlib#18198 – `FunLike` change to `Data.PEquiv` is already in mathlib4.

Substantative forward port:
* leanprover-community/mathlib#18520



Co-authored-by: Parcly Taxel <reddeloostw@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated The PR author may merge after reviewing final suggestions. ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) t-differential-geometry Manifolds, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants