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: structure sheaf of a manifold #7332

Closed
wants to merge 23 commits into from

Conversation

hrmacbeth
Copy link
Member

@hrmacbeth hrmacbeth commented Sep 23, 2023

In leanprover-community/mathlib#19146, we defined StructureGroupoid.LocalInvariantProp.sheaf, the sheaf-of-types of functions f : M → M' (for charted spaces M, M') satisfying some local property in the sense of StructureGroupoid.LocalInvariantProp (for example continuity, differentiability, smoothness).

In this PR, in the case of smoothness, we upgrade this to a sheaf of groups if M' is a Lie group, a sheaf of abelian groups if M' is an abelian Lie group, and a sheaf of rings if M' is a "smooth ring".

Co-authored-by: Adam Topaz github@adamtopaz.com


Open in Gitpod

This PR replaces the mathlib3 PR leanprover-community/mathlib#19094, which I will now close.

A few things had to be manually to-additivized (and had not needed to be in Lean 3), expert help welcome! fixed by Floris

@hrmacbeth hrmacbeth added awaiting-review The author would like community review of the PR t-differential-geometry Manifolds etc labels Sep 26, 2023
Copy link
Contributor

@ocfnash ocfnash left a comment

Choose a reason for hiding this comment

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

Just a few nitpicks: let's get this merged!

I also looked over the conversation at leanprover-community/mathlib#19094 and I note that there is nothing pending from there (except the declared-out-of-scope approach of "interpreting Lawvere theories in categories with finite products" to avoid repetition).

bors d+

Mathlib/Topology/Category/TopCat/Basic.lean Outdated Show resolved Hide resolved
Comment on lines 54 to 57
obstacle here is in the underlying category theory constructions, and there is WIP (as of June 2023)
to fix this. See
https://github.com/leanprover-community/mathlib/pull/19153
and cross-references there.
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be nice to give a bit more detail here.

I looked at leanprover-community/mathlib#19153 and there are some useful references but I didn't really feel this enhanced my understanding beyond "some parts of the category theory library are probably not sufficiently universe polymorphic".

Copy link
Member Author

@hrmacbeth hrmacbeth Oct 7, 2023

Choose a reason for hiding this comment

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

a7d51dd updates to what I think is the current status here. @semorrison Could you look at that diff to see whether it's an accurate description?

edit: ignore the incorrect year on that diff -- I have fixed it now!

Copy link
Contributor

Choose a reason for hiding this comment

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

LGTM.

Mathlib/Geometry/Manifold/Sheaf/Smooth.lean Outdated Show resolved Hide resolved
Mathlib/Geometry/Manifold/Sheaf/Smooth.lean Outdated Show resolved Hide resolved
Mathlib/Geometry/Manifold/Sheaf/Smooth.lean Outdated Show resolved Hide resolved
Mathlib/Geometry/Manifold/Sheaf/Smooth.lean Outdated Show resolved Hide resolved
Mathlib/Geometry/Manifold/Sheaf/Smooth.lean Outdated Show resolved Hide resolved
@bors
Copy link

bors bot commented Oct 5, 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-mathlib4-bot leanprover-community-mathlib4-bot added delegated and removed awaiting-review The author would like community review of the PR labels Oct 5, 2023
@fpvandoorn
Copy link
Member

To fix the to_additive issues, we need:

attribute [to_additive existing TopologicalSpace] TopologicalSpace

(this is to enable/work around the heuristics of to_additive - this should probably be done in TopologicalSpace.Basic)

noncomputable in front of every declaration that you additivize. This is issue leanprover/lean4#2610

See: b69521d

@hrmacbeth
Copy link
Member Author

bors r+

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Oct 9, 2023
bors bot pushed a commit that referenced this pull request Oct 9, 2023
In leanprover-community/mathlib#19146, we defined `StructureGroupoid.LocalInvariantProp.sheaf`, the sheaf-of-types of functions `f : M → M'` (for charted spaces `M`, `M'`) satisfying some local property in the sense of `StructureGroupoid.LocalInvariantProp` (for example continuity, differentiability, smoothness).

In this PR, in the case of smoothness, we upgrade this to a sheaf of groups if `M'` is a Lie group, a sheaf of abelian groups if `M'` is an abelian Lie group, and a sheaf of rings if `M'` is a "smooth ring".

Co-authored-by: Adam Topaz <github@adamtopaz.com>



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

bors bot commented Oct 9, 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 feat: structure sheaf of a manifold [Merged by Bors] - feat: structure sheaf of a manifold Oct 9, 2023
@bors bors bot closed this Oct 9, 2023
@bors bors bot deleted the HM-smooth-sheaf branch October 9, 2023 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated ready-to-merge This PR has been sent to bors. t-differential-geometry Manifolds etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants