Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(CategoryTheory/Monoidal): add lemmas for whiskering operators #8191

Merged
merged 2 commits into from
Nov 7, 2023

Conversation

yuma-mizuno
Copy link
Collaborator

Extracted from #6307 as suggested in #6307 (comment) .


Open in Gitpod

@joelriou
Copy link
Collaborator

joelriou commented Nov 5, 2023

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 5, 2023
@@ -509,6 +509,16 @@ theorem tensorHom {W X Y Z : Action V G} (f : W ⟶ X) (g : Y ⟶ Z) : (f ⊗ g)
set_option linter.uppercaseLean3 false in
#align Action.tensor_hom Action.tensorHom

@[simp]
theorem whiskerLeft_v (X : Action V G) {Y Z : Action V G} (f : Y ⟶ Z) :
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure the v is appropriate here, I'd expect hom

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks! I also renamed tensorHom to tensor_hom (probably a remnant from porting).

@eric-wieser
Copy link
Member

bors merge

(I think bors is broken, but this will flag the PR as ready for when we sort it out)

Thanks!

@robertylewis robertylewis added this pull request to the merge queue Nov 5, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 5, 2023
@robertylewis robertylewis added this pull request to the merge queue Nov 5, 2023
@robertylewis robertylewis removed this pull request from the merge queue due to a manual request Nov 5, 2023
@robertylewis robertylewis added this pull request to the merge queue Nov 5, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 5, 2023
@sgouezel sgouezel added this pull request to the merge queue Nov 7, 2023
Merged via the queue into master with commit 64b26c9 Nov 7, 2023
19 checks passed
grunweg pushed a commit that referenced this pull request Dec 15, 2023
…8191)

Extracted from #6307 as suggested in
#6307 (comment)
.

---
<!-- The text above the `---` will become the commit message when your
PR is merged. Please leave a blank newline before the `---`, otherwise
GitHub will format the text above it as a title.

To indicate co-authors, include lines at the bottom of the commit
message
(that is, before the `---`) using the following format:

Co-authored-by: Author Name <[email protected]>

Any other comments you want to keep out of the PR commit should go
below the `---`, and placed outside this HTML comment, or else they
will be invisible to reviewers.

If this PR depends on other PRs, please list them below this comment,
using the following format:
- [ ] depends on: #abc [optional extra text]
- [ ] depends on: #xyz [optional extra text]
-->

[![Open in
Gitpod](https://gitpod.io/button/open-in-gitpod.svg)](https://gitpod.io/from-referrer/)
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.

4 participants