Skip to content

[Merged by Bors] - feat(Condensed): colimit characterization of discrete condensed sets#15566

Closed
dagurtomas wants to merge 96 commits intomasterfrom
dagur/CondensedDiscreteColimit
Closed

[Merged by Bors] - feat(Condensed): colimit characterization of discrete condensed sets#15566
dagurtomas wants to merge 96 commits intomasterfrom
dagur/CondensedDiscreteColimit

Conversation

@dagurtomas
Copy link
Copy Markdown
Contributor

@dagurtomas dagurtomas commented Aug 6, 2024

@jcommelin
Copy link
Copy Markdown
Member

What do you think of removing the t-category-theory label? Since t-condensed is an "official" top-level label.

@dagurtomas dagurtomas removed the t-category-theory Category theory label Sep 27, 2024
@leanprover-community-bot-assistant leanprover-community-bot-assistant added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Oct 3, 2024
@dagurtomas dagurtomas removed the awaiting-author A reviewer has asked the author a question or requested changes. label Oct 4, 2024
@leanprover-community-bot-assistant leanprover-community-bot-assistant removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Oct 4, 2024
@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Oct 4, 2024
@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Oct 4, 2024
The functor `locallyConstantPresheaf` takes sequential limits of finite sets with surjective
projection maps to colimits.
-/
noncomputable def isColimitLocallyConstantPresheaf (hc : IsLimit c) [∀ i, Epi (c.π.app i)] :
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should this be accompanied by simp lemmas?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think we rarely have simp lemmas accompanying defs of the type IsColimit, because it makes you lose access to the nice general IsColimit API. In this case it would have the form (isColimitLocallyConstantPresheaf c X hc).desc s x = s.ι.app ⋯.choose ⋯.choose, which isn't very helpful

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I've added lemmas below that are less general than the ones generated by simps, but which might be useful to have.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, and fair point.

Copy link
Copy Markdown
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

Thanks 🎉

bors merge

@ghost ghost added the ready-to-merge This PR has been sent to bors. label Oct 16, 2024
mathlib-bors bot pushed a commit that referenced this pull request Oct 16, 2024
…15566)

This PR provides the necessary API to prove that a condensed set `X` is discrete if and only if
for every profinite set `S = limᵢSᵢ`, `X(S) ≅ colimᵢX(Sᵢ)`, and the analogous result for light
condensed sets.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Oct 16, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Condensed): colimit characterization of discrete condensed sets [Merged by Bors] - feat(Condensed): colimit characterization of discrete condensed sets Oct 16, 2024
@mathlib-bors mathlib-bors bot closed this Oct 16, 2024
@mathlib-bors mathlib-bors bot deleted the dagur/CondensedDiscreteColimit branch October 16, 2024 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. t-condensed Condensed mathematics

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants