Skip to content

refactor(CategoryTheory/Monoidal): add whiskering operators#6307

Draft
yuma-mizuno wants to merge 79 commits intomasterfrom
ymizuno-monoidal-whiskering
Draft

refactor(CategoryTheory/Monoidal): add whiskering operators#6307
yuma-mizuno wants to merge 79 commits intomasterfrom
ymizuno-monoidal-whiskering

Conversation

@yuma-mizuno
Copy link
Copy Markdown
Collaborator

@yuma-mizuno yuma-mizuno commented Aug 2, 2023

@yuma-mizuno yuma-mizuno added the WIP Work in progress label Aug 2, 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 Aug 2, 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 3, 2023
kim-em and others added 3 commits August 5, 2023 20:27
@yuma-mizuno yuma-mizuno added awaiting-review and removed WIP Work in progress labels Aug 5, 2023
@yuma-mizuno yuma-mizuno changed the title refactor(CategoryTheory/Monoidal): whiskerings as fundamental refactor(CategoryTheory/Monoidal): add whiskering operators Aug 5, 2023
@kim-em kim-em added the t-category-theory Category theory label Aug 6, 2023
Comment on lines +548 to +554
calc
_ = 𝟙 _ ⊗≫
X₁ ◁ ((β_ X₂ Y₁).hom ▷ (Y₂ ⊗ Z₁) ≫ (Y₁ ⊗ X₂) ◁ (β_ Y₂ Z₁).hom) ▷ Z₂ ⊗≫
X₁ ◁ Y₁ ◁ (β_ X₂ Z₁).hom ▷ Y₂ ▷ Z₂ ⊗≫ 𝟙 _ := ?eq1
_ = _ := ?eq2
case eq1 => coherence
case eq2 => rw [← whisker_exchange]; coherence
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.

My preference is not to use this ?eqn style, but to write the proofs inline in the calc block, at least when the proofs are as short as these ones.

I'm happy to use this technique for multiline proofs needed in a calc block, however.

mathlib-bors bot pushed a commit that referenced this pull request Feb 29, 2024
riccardobrasca pushed a commit that referenced this pull request Mar 1, 2024
mathlib-bors bot pushed a commit that referenced this pull request Mar 1, 2024
We set `id_tensorHom` and `tensorHom_id` as simp lemmas. Partially extracted from #6307.
mathlib-bors bot pushed a commit that referenced this pull request Mar 1, 2024
We set `id_tensorHom` and `tensorHom_id` as simp lemmas. Partially extracted from #6307.
mathlib-bors bot pushed a commit that referenced this pull request Mar 1, 2024
We set `id_tensorHom` and `tensorHom_id` as simp lemmas. Partially extracted from #6307.
@ghost ghost removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Mar 1, 2024
@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 Mar 2, 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 Mar 4, 2024
Cones.postcompose_obj_π, NatTrans.comp_app, Functor.const_obj_obj, Monoidal.tensorObj_obj,
Monoidal.tensorUnit_obj, Monoidal.leftUnitor_hom_app]
conv_rhs => rw [← tensor_id_comp_id_tensor (limit.π X j)]
simp
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[lint-style] reported by reviewdog 🐶

Suggested change
simp
simp?

Cones.postcompose_obj_π, NatTrans.comp_app, Functor.const_obj_obj, Monoidal.tensorObj_obj,
Monoidal.tensorUnit_obj, Monoidal.rightUnitor_hom_app]
conv_rhs => rw [← id_tensor_comp_tensor_id _ (limit.π X j)]
simp
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[lint-style] reported by reviewdog 🐶

Suggested change
simp
simp?

@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 Mar 4, 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 Mar 4, 2024
@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 Mar 7, 2024
@ghost ghost added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Mar 7, 2024
mathlib-bors bot pushed a commit that referenced this pull request Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-author A reviewer has asked the author a question or requested changes. merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) t-category-theory Category theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants