Skip to content

[Merged by Bors] - feat(CategoryTheory): inductive construction of composable arrows#8146

Closed
joelriou wants to merge 6 commits intomasterfrom
composable-arrows-3
Closed

[Merged by Bors] - feat(CategoryTheory): inductive construction of composable arrows#8146
joelriou wants to merge 6 commits intomasterfrom
composable-arrows-3

Conversation

@joelriou
Copy link
Copy Markdown
Contributor

@joelriou joelriou commented Nov 3, 2023


Open in Gitpod

@joelriou joelriou added the t-category-theory Category theory label Nov 3, 2023
@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 Nov 3, 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 Nov 3, 2023
Comment on lines +420 to +428
namespace Nerve

variable {C}

lemma δ₀_eq_composableArrows_δ₀ {n : ℕ}
(x : (nerve C).obj (Opposite.op (SimplexCategory.mk (n + 1)))) :
(nerve C).δ (0 : Fin (n + 2)) x = ComposableArrows.δ₀ x := rfl

end Nerve
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.

I wonder whether we should revert the import. If so, then we can do that in another PR.

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.

@jcommelin Do you mean have Nerve import ComposableArrows rather than the other direction, so that we may use ComposableArrows without relying on the whole simplicial stuff?

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.

Exactly! And it would also allow us to use ComposableArrows from the start in setting up API for Nerve.

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.

As bors failed, I shall do it in this PR.

Copy link
Copy Markdown
Contributor Author

@joelriou joelriou Nov 3, 2023

Choose a reason for hiding this comment

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

This is done in 6a9f235 (I had to introduce some basic functoriality of ComposableArrows).

@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 Nov 3, 2023
@bors
Copy link
Copy Markdown

bors bot commented Nov 3, 2023

Build failed (retrying...):

@jcommelin
Copy link
Copy Markdown
Member

bors d+

@bors
Copy link
Copy Markdown

bors bot commented Nov 3, 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 the delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). label Nov 3, 2023
@bors
Copy link
Copy Markdown

bors bot commented Nov 3, 2023

Canceled.

@joelriou joelriou added awaiting-review and removed 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. labels Nov 3, 2023
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 ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Nov 3, 2023
@ocfnash
Copy link
Copy Markdown
Contributor

ocfnash commented Nov 3, 2023

GitHub was flaking and bors crashed so we need to try this again

bors merge

@bors
Copy link
Copy Markdown

bors bot commented Nov 3, 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): inductive construction of composable arrows [Merged by Bors] - feat(CategoryTheory): inductive construction of composable arrows Nov 3, 2023
@bors bors bot closed this Nov 3, 2023
@bors bors bot deleted the composable-arrows-3 branch November 3, 2023 21:31
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants