Skip to content

[Merged by Bors] - feat: projectivity of Stonean in CompHaus #5808

Closed
dagurtomas wants to merge 32 commits intomasterfrom
ExtrDisc2
Closed

[Merged by Bors] - feat: projectivity of Stonean in CompHaus #5808
dagurtomas wants to merge 32 commits intomasterfrom
ExtrDisc2

Conversation

@dagurtomas
Copy link
Copy Markdown
Contributor

@dagurtomas dagurtomas commented Jul 11, 2023

Co-authored-by: Filippo A E Nuccio filippo.nuccio@univ-st-etienne.fr
Co-authored-by: Nikolas Kuhn nikolaskuhn@gmx.de
Co-authored-by: Adam Topaz topaz@ualberta.ca

This work was done during the 2023 Copenhagen masterclass on formalisation of condensed mathematics. Numerous participants contributed.


Open in Gitpod

@dagurtomas dagurtomas added the WIP Work in progress label Jul 11, 2023
@dagurtomas dagurtomas changed the title More ExtrDisc feat: projectivity of ExtrDisc in CompHaus Jul 11, 2023
@dagurtomas dagurtomas added awaiting-review awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. and removed WIP Work in progress labels Jul 11, 2023
@kim-em kim-em added the t-category-theory Category theory label Aug 6, 2023
@ghost ghost removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Aug 9, 2023
@ghost ghost deleted a comment from kim-em Aug 9, 2023
@ghost
Copy link
Copy Markdown

ghost commented Aug 9, 2023

@ghost ghost removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Aug 22, 2023
@dagurtomas dagurtomas added awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. t-topology Topological spaces, uniform spaces, metric spaces, filters labels Aug 22, 2023
@github-actions github-actions bot removed the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Aug 22, 2023
@dagurtomas dagurtomas changed the title feat: projectivity of ExtrDisc in CompHaus feat: projectivity of Stonean in CompHaus Aug 22, 2023

/-- The morphism from `presentation X` to `X`. -/
noncomputable
def presentationπ (X : CompHaus) : X.presentation.compHaus ⟶ 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.

What do you think of chopping the name into an extra namespace part?

Suggested change
def presentationπ (X : CompHaus) : X.presentation.compHaus ⟶ X :=
def presentation.π (X : CompHaus) : X.presentation.compHaus ⟶ X :=

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 agree that looks nicer

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.

When I change to presentation.π, I can't write X.presentation.π in the instance below anymore. It complains that "the environment does not contain Stonean.π, presentation X has type Stonean".

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.

No permutation of parentheses fixes the problem

Copy link
Copy Markdown
Contributor Author

@dagurtomas dagurtomas Aug 22, 2023

Choose a reason for hiding this comment

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

We could go with presentation_π and presentation_epi_π which I think is still nicer than the old names.

Or we can still go with your suggestion and then write the instance below as

instance presentation.epi_π (X : CompHaus) : Epi (π X)

What do you think?

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 pushed your suggestion, but I'm happy with whatever

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.

Let's try it out like this. We can rename again later if we are unhappy.


/-- The morphism from `presentation X` to `X` is an epimorphism. -/
noncomputable
instance epiPresentπ (X : CompHaus) : Epi X.presentationπ :=
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.

If you agree with the above suggestion, then this might become

Suggested change
instance epiPresentπ (X : CompHaus) : Epi X.presentationπ :=
instance presentation.epi_π (X : CompHaus) : Epi X.presentationπ :=

@jcommelin
Copy link
Copy Markdown
Member

Thanks 🎉

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Aug 22, 2023
bors bot pushed a commit that referenced this pull request Aug 22, 2023
Co-authored-by: Filippo A E Nuccio [filippo.nuccio@univ-st-etienne.fr](mailto:filippo.nuccio@univ-st-etienne.fr)
Co-authored-by: Nikolas Kuhn [nikolaskuhn@gmx.de](mailto:nikolaskuhn@gmx.de)
Co-authored-by: Adam Topaz [topaz@ualberta.ca](mailto:topaz@ualberta.ca)

 -  [x] depends on: #5634 
 -  [x] depends on: #5761  

This work was done during the 2023 Copenhagen masterclass on formalisation of condensed mathematics. Numerous participants contributed.



Co-authored-by: David Kurniadi Angdinata <dka31@cantab.ac.uk>
@bors
Copy link
Copy Markdown

bors bot commented Aug 22, 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: projectivity of Stonean in CompHaus [Merged by Bors] - feat: projectivity of Stonean in CompHaus Aug 22, 2023
@bors bors bot closed this Aug 22, 2023
@bors bors bot deleted the ExtrDisc2 branch August 22, 2023 17:50
dagurtomas added a commit that referenced this pull request Aug 22, 2023
dagurtomas added a commit that referenced this pull request Aug 22, 2023
dagurtomas added a commit that referenced this pull request Aug 22, 2023
bors bot pushed a commit that referenced this pull request Aug 23, 2023
Co-authored-by: Jon Eugster @joneugster  
Co-authored-by: Boris Bolvig Kjær <bbk@math.ku.dk> @bbolvig
Co-authored-by: Sina Hazratpour <sinahazratpour@gmail.com> @sinhp
Co-authored-by: Nima Rasekh <rasekh@mpim-bonn.mpg.de> @nimarasekh 

 -  [x] depends on: #5808  

We give a characterisation of effective epimorphic families in `Stonean` and deduce that `Stonean` is a precoherent category.
This work was done during the 2023 Copenhagen masterclass on formalisation of condensed mathematics. Numerous participants contributed.
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-category-theory Category theory t-topology Topological spaces, uniform spaces, metric spaces, filters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants