Skip to content

[Merged by Bors] - feat(Condensed): characterisation of discrete (light) condensed sets and modules#14027

Closed
dagurtomas wants to merge 382 commits intomasterfrom
dagur/DiscreteCondensedObjects
Closed

[Merged by Bors] - feat(Condensed): characterisation of discrete (light) condensed sets and modules#14027
dagurtomas wants to merge 382 commits intomasterfrom
dagur/DiscreteCondensedObjects

Conversation

@dagurtomas
Copy link
Copy Markdown
Contributor

@dagurtomas dagurtomas commented Jun 22, 2024

This PR provides a characterization of discrete condensed sets and modules, both in the light setting and the classical one.

  • Six equivalent conditions on a condensed set X to be discrete
    1. The counit of the discrete-underlying adjunction applied to X is an isomorphism
    2. X is in the essential image of the constant sheaf functor Type (u+1) ⥤ CondensedSet.{u}
    3. X is in the essential image of the functor Type (u+1) ⥤ CondensedSet.{u} which takes a set to the sheaf of locally constant maps into it.
    4. The counit of the locally-constant-underlying adjunction applied to X is an isomorphism.
    5. X restricted to the coherent site of profinite sets is discrete as a sheaf
    6. For every profinite set S written as a limit of finite sets, X maps S to the corresponding colimit.
  • A condensed module over a ring is discrete if and only if its underlying condensed set is, and therefore the analogues of the equivalent conditions above also characterize condensed modules as discrete
  • The analogues for the above for light condensed sets and modules

Open in Gitpod

@dagurtomas dagurtomas added the WIP Work in progress label Jun 22, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 22, 2024

PR summary 8f7896ed23

Import changes exceeding 2%

% File
+4.47% Mathlib.Algebra.Category.ModuleCat.FilteredColimits

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.Algebra.Category.ModuleCat.FilteredColimits 805 841 +36 (+4.47%)
Import changes for all files
Files Import difference
Mathlib.Algebra.Category.ModuleCat.FilteredColimits 36
Mathlib.Condensed.Discrete.Characterization 1696

Declarations diff

+ forgetReflectsFilteredColimits
+ instance : HasLimitsOfSize.{u, u+1} (ModuleCat.{u+1} R)
++ IsDiscrete
++ LocallyConstant.adjunction
++ isDiscrete_iff_isDiscrete_forget
++ mem_locallyContant_essImage_of_isColimit_mapCocone
++++ isDiscrete_tfae

You can run this locally as follows
## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>

The doc-module for script/declarations_diff.sh contains some details about this script.

@dagurtomas dagurtomas changed the title feat(Condensed): (light) condensed modules are discrete iff the underlying condensed set is feat(Condensed): characterisation of discrete (light) condensed sets and modules Jun 22, 2024
@ghost ghost added merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) and removed merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) labels Jun 26, 2024
@ghost ghost added merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) and removed merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) labels Jul 2, 2024
@dagurtomas dagurtomas removed the WIP Work in progress label Jul 3, 2024
@ghost ghost added merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) and removed merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) labels Jul 4, 2024
@dagurtomas dagurtomas added t-category-theory Category theory t-condensed Condensed mathematics labels Jul 10, 2024
@ghost ghost added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jul 10, 2024
@ghost ghost added merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) and removed merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) labels Jul 18, 2024
@ghost ghost added merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) and removed merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) labels Jul 23, 2024
@github-actions github-actions bot added the large-import Automatically added label for PRs with a significant increase in transitive imports label Oct 8, 2024
@dagurtomas
Copy link
Copy Markdown
Contributor Author

dagurtomas commented Oct 8, 2024

PR summary 92528ad914

Import changes exceeding 2%

% File
+4.37% Mathlib.Algebra.Category.ModuleCat.FilteredColimits

Import changes for modified files

Declarations diff

I think it's completely reasonable that the file ModuleCat.FilteredColimits knows that ModuleCat has colimits

@dagurtomas dagurtomas removed the t-category-theory Category theory label Oct 8, 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 16, 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 16, 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 17, 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 17, 2024
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.

Very nice! Congratulations with reaching this milestone 😃

bors d+

[ X.IsDiscrete
, IsIso ((Condensed.discreteUnderlyingAdj _).counit.app X)
, X ∈ (Condensed.discrete _).essImage
, X ∈ functor.essImage
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.

Since this is the "entry point" for using all these equivalent results, I suggest spelling out the full name of functor. Just to increase the readability of this particular statement.
Idem dito for adjunction below.

Comment on lines +198 to +199
, X ∈ functor.essImage
, IsIso (adjunction.counit.app X)
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.

Idem

Comment on lines +235 to +236
, M ∈ (functor R).essImage
, IsIso ((adjunction R).counit.app M)
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.

Idem

Comment on lines +126 to +127
, M ∈ (functor R).essImage
, IsIso ((adjunction R).counit.app M)
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.

Idem

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Oct 17, 2024

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

@ghost ghost added the delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). label Oct 17, 2024
@dagurtomas dagurtomas 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 17, 2024
@dagurtomas
Copy link
Copy Markdown
Contributor Author

Thanks for all the reviews 🎉

bors merge

mathlib-bors bot pushed a commit that referenced this pull request Oct 17, 2024
…and modules (#14027)

This PR provides a characterization of discrete condensed sets and modules, both in the light setting and the classical one. 

* Six equivalent conditions on a condensed set X to be discrete
  1. The counit of the discrete-underlying adjunction applied to X is an isomorphism
  2. X is in the essential image of the constant sheaf functor `Type (u+1) ⥤ CondensedSet.{u}`
  3. X is in the essential image of the functor `Type (u+1) ⥤ CondensedSet.{u}` which takes a set to the sheaf of locally             constant maps into it.
  4. The counit of the locally-constant-underlying adjunction applied to X is an isomorphism.
  5. X restricted to the coherent site of profinite sets is discrete as a sheaf
  6. For every profinite set S written as a limit of finite sets, X maps S to the corresponding colimit.
* A condensed module over a ring is discrete if and only if its underlying condensed set is, and therefore the analogues of the equivalent conditions above also characterize condensed modules as discrete
* The analogues for the above for light condensed sets and modules
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Oct 17, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Condensed): characterisation of discrete (light) condensed sets and modules [Merged by Bors] - feat(Condensed): characterisation of discrete (light) condensed sets and modules Oct 17, 2024
@mathlib-bors mathlib-bors bot closed this Oct 17, 2024
@mathlib-bors mathlib-bors bot deleted the dagur/DiscreteCondensedObjects branch October 17, 2024 08:51
mathlib-bors bot pushed a commit that referenced this pull request Oct 24, 2024
These are not relevant now that #14027 has been merged.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). large-import Automatically added label for PRs with a significant increase in transitive imports t-condensed Condensed mathematics

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants