-
Notifications
You must be signed in to change notification settings - Fork 67
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
Extremal Morphisms #340
Extremal Morphisms #340
Conversation
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.
Nice.
A couple of minor things (no need to change for this PR):
- it really is best to use
using
as much as possible on allopen import
- I really would have preferred 2 PRs, one for the clean-up, and one for the new functionality.
Can you open an issue about the naming of the fields in |
Opened #341 to track the naming issue. I'll leave this up for a few days to get feedback, but if no one has anything to say I think we should be good to merge :) |
Patch Description
This PR implements Extremal Morphisms, and proves some properties about them. It also proves some helper lemmas about epis/monos, and performs some minor cleanup of equalizers + coequalizers.
Notes
The names of the morphisms in
RegularMono
/RegularEpi
are really annoying... Perhaps there's a better choice instead ofg
andh
?