Skip to content

[Merged by Bors] - feat(CategoryTheory): calculus of fractions#8041

Closed
joelriou wants to merge 6 commits intomasterfrom
calculusoffractions1
Closed

[Merged by Bors] - feat(CategoryTheory): calculus of fractions#8041
joelriou wants to merge 6 commits intomasterfrom
calculusoffractions1

Conversation

@joelriou
Copy link
Copy Markdown
Contributor

@joelriou joelriou commented Oct 30, 2023

This PR introduces notions of left and right fractions for a morphism property in a category. The typeclasses MorphismProperty.HasLeftCalculusOfFractions and MorphismProperty.HasRightCalculusOfFractions are also introduced.


Open in Gitpod

@joelriou joelriou added WIP Work in progress t-category-theory Category theory labels Oct 30, 2023
@joelriou joelriou added awaiting-review and removed WIP Work in progress labels Oct 30, 2023
have := hL _ φ.hs
L.map φ.f ≫ inv (L.map φ.s)

@[reassoc (attr := simp, nolint unusedHavesSuffices)]
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.

This nolint points at a bug. Is there already an issue for that?

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.

Using letI instead of have fixed the problem...

@jcommelin
Copy link
Copy Markdown
Member

Thanks 🎉

If CI passes, please remove the label awaiting-CI and merge this yourself, by adding a comment bors r+.

bors d+

@bors
Copy link
Copy Markdown

bors bot commented Oct 30, 2023

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

@github-actions github-actions bot added delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). and removed awaiting-review labels Oct 30, 2023
@joelriou
Copy link
Copy Markdown
Contributor Author

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Oct 30, 2023
Comment on lines +180 to +181
ext : ∀ ⦃X' X Y : C⦄ (f₁ f₂ : X ⟶ Y) (s : X' ⟶ X) (_ : W s)
(_ : s ≫ f₁ = s ≫ f₂), ∃ (Y' : C) (t : Y ⟶ Y') (_ : W t), f₁ ≫ t = f₂ ≫ t
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why not

Suggested change
ext : ∀ ⦃X' X Y : C⦄ (f₁ f₂ : X ⟶ Y) (s : X' ⟶ X) (_ : W s)
(_ : s ≫ f₁ = s ≫ f₂), ∃ (Y' : C) (t : Y ⟶ Y') (_ : W t), f₁ ≫ t = f₂ ≫ t
ext : ∀ ⦃X' X Y : C⦄ (f₁ f₂ : X ⟶ Y) (s : X' ⟶ X),
W s → s ≫ f₁ = s ≫ f₂∃ (Y' : C) (t : Y ⟶ Y'), W t f₁ ≫ t = f₂ ≫ t


namespace MorphismProperty

variable {C D : Type _} [Category C] [Category D]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think Type* is preferred now

bors bot pushed a commit that referenced this pull request Oct 30, 2023
This PR introduces notions of left and right fractions for a morphism property in a category. The typeclasses `MorphismProperty.HasLeftCalculusOfFractions` and `MorphismProperty.HasRightCalculusOfFractions` are also introduced.
@bors
Copy link
Copy Markdown

bors bot commented Oct 30, 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(CategoryTheory): calculus of fractions [Merged by Bors] - feat(CategoryTheory): calculus of fractions Oct 30, 2023
@bors bors bot closed this Oct 30, 2023
@bors bors bot deleted the calculusoffractions1 branch October 30, 2023 20:20
grunweg pushed a commit that referenced this pull request Nov 1, 2023
This PR introduces notions of left and right fractions for a morphism property in a category. The typeclasses `MorphismProperty.HasLeftCalculusOfFractions` and `MorphismProperty.HasRightCalculusOfFractions` are also introduced.
fgdorais pushed a commit that referenced this pull request Nov 1, 2023
This PR introduces notions of left and right fractions for a morphism property in a category. The typeclasses `MorphismProperty.HasLeftCalculusOfFractions` and `MorphismProperty.HasRightCalculusOfFractions` are also introduced.
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). ready-to-merge This PR has been sent to bors. t-category-theory Category theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants