Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

[Merged by Bors] - feat(geometry/manifold/vector_bundle/basic): smooth vector bundles#17611

Closed
hrmacbeth wants to merge 50 commits intomasterfrom
smooth-vector-bundle-pr
Closed

[Merged by Bors] - feat(geometry/manifold/vector_bundle/basic): smooth vector bundles#17611
hrmacbeth wants to merge 50 commits intomasterfrom
smooth-vector-bundle-pr

Conversation

@hrmacbeth
Copy link
Copy Markdown
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
Copy Markdown
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
@fpvandoorn fpvandoorn requested a review from sgouezel February 28, 2023 19:18
@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
Copy Markdown
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+


## Main definitions and constructions

* `fiber_bundle.charted_space`: A fibre bundle `E` over a base `B` with model fibre `F` is naturally
Copy link
Copy Markdown
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.

@bors
Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown

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/mathlib3#17611 – `CategoryTheory.Sites.Sheaf` already uses `aesop_cat` at this location, nothing to port.
* leanprover-community/mathlib3#18742 – fiber spelling, which is all of the changes to `Topology.FiberBundle.Trivialization`, is already in mathlib4.
* leanprover-community/mathlib3#18198 – `FunLike` change to `Data.PEquiv` is already in mathlib4.

Substantative forward port:
* leanprover-community/mathlib3#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/mathlib3#17611 – `CategoryTheory.Sites.Sheaf` already uses `aesop_cat` at this location, nothing to port.
* leanprover-community/mathlib3#18742 – fiber spelling, which is all of the changes to `Topology.FiberBundle.Trivialization`, is already in mathlib4.
* leanprover-community/mathlib3#18198 – `FunLike` change to `Data.PEquiv` is already in mathlib4.

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



Co-authored-by: Parcly Taxel <reddeloostw@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

4 participants