-
Notifications
You must be signed in to change notification settings - Fork 195
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
Update bifunctors #1886
Merged
Merged
Update bifunctors #1886
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
63e3847
Move bifunctor coherence to Is1Bifunctor
gio256 783f5b9
WildCat/Bifunctor.v: Remove bifunctor_hom lemmas
gio256 a95509d
AbSES/BaerSum.v: fix proof of is1bifunctor_abses
gio256 1046b48
WildCat/Bifunctor.v: simplify uncurried proofs
gio256 62b8d2b
AbGroups/AbHom.v: prove is1bifunctor_ab_hom
gio256 720075a
AbSES/Ext.v: rough proofs of Is1Bifunctor
gio256 56d82e7
AbSES/Ext.v: simplify bifunctor proofs for ab_ext
gio256 2c9fce6
AbSES/Ext.v: remove universe annotations
gio256 d78a876
Simplify building of bifunctors
gio256 e19d81e
WildCat/Yoneda.v: Add bifunctor instances for hom
gio256 f5e4d86
WildCat/Prod.v: add product inclusions
gio256 192a786
WildCat/Bifunctor.v: remove comment
gio256 beed14d
AbSES/Ext.v: remove universe annotations
gio256 f930232
WildCat/Yoneda.v: add comment
gio256 da5cb5a
WildCat/Monoidal: tensor product is a 1-bifunctor
gio256 c41d397
WildCat equivs: add compose_catie'
gio256 2ae47ce
Move bifunctor lemmas from Prod.v to Bifunctor.v
gio256 1426fd9
WildCat/Bifunctor.v: clean up two proofs
gio256 b662a72
WildCat equivs: re-define compose_catie'
gio256 f902579
WildCat equivs fix: cate_homotopic to catie_homotopic
gio256 c2616da
WildCat/Yoneda.v: opyon_0gpd bifunctor instances
gio256 07b6ff4
contrib/SetoidRewrite.v: formatting
gio256 2ed1065
WildCat/Yoneda.v: clean up is0functor_hom_0gpd
gio256 6c975c1
WildCat/Bifunctor.v: formatting
gio256 31562d0
WildCat/Yoneda.v: add comment
gio256 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you also prove the 0gpd versions which appear further down? Those won't require
MorphismExtensionality
IIUC.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you think about proving
is?functor_opyon
using the bifunctor instances instead of the explicit proof that is there now? It seems like a repeated proof, but maybe an instructive one.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not certain. I think for now lets keep it repeated and merge it at another point if we think it would be slicker.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe @jdchristensen will have an opinion when he is available.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know exactly what is being discussed, but I'm all in favour of reusing proofs instead of repeating them (if it works smoothly).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried proving
is?functor_opyon
using the bifunctor instances, but it leads to a proliferation of extra identity maps in inconvenient places. I will play with it some more, but there might be a reason these weren't proven from theis?functor_hom
instances previously, in which case maybe best just to leave a comment explaining why it is proven explicitly.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have added a comment for this now, though I debated leaving no comment instead. It's a bit misleading, because there are really a number of reasons that we prove
is0functor_opyon
explicitly:is0functor_yon
asis0functor_opyon (A:=A^op)
, which is consistent with how the rest of the contravariant lemmas are proven.