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

derived ImageEmbedding as the colift along the coastriction to image #1495

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mohamed-barakat
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Oct 17, 2023

Codecov Report

Attention: 6 lines in your changes are missing coverage. Please review.

Files Coverage Δ
CAP/PackageInfo.g 100.00% <100.00%> (ø)
CAP/gap/DerivedMethods.autogen.gi 86.93% <88.88%> (+0.03%) ⬆️
CAP/gap/DerivedMethods.gi 84.00% <71.42%> (-0.11%) ⬇️

📢 Thoughts on this report? Let us know!.

@zickgraf
Copy link
Member

Please also add a WithGiven version of the derivation to make it consistent with the analogous derivations of CoastrictionToImage(WithGiven...).

@zickgraf
Copy link
Member

I just noticed that CAP's definition of Image does not require the coastriction to the image to be epi. Hence, in general this derivation is not valid, or am I missing something?

@mohamed-barakat
Copy link
Member Author

I just noticed that CAP's definition of Image does not require the coastriction to the image to be epi. Hence, in general this derivation is not valid, or am I missing something?

You are right, I have to use the general lift an colift.

@mohamed-barakat
Copy link
Member Author

You are right, I have to use the general lift an colift.

Such a factorization is always unique up to unique isomorphism.

@mohamed-barakat
Copy link
Member Author

It is now unclear to me why the colift is necessarily a monomorphism.

@mohamed-barakat
Copy link
Member Author

I restricted the derivation to IsAbelianCategory. I remember that the natural morphism from the coimage to image is both mono and epi, making these derivations also valid for IsPreAbelianCategory. I can relax the conditions in a later PR once I have a proof.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants