CAP_project icon indicating copy to clipboard operation
CAP_project copied to clipboard

The role of `CanonicalIdentificationFromCoimageToImageObject`

Open zickgraf opened this issue 2 years ago • 3 comments

CanonicalIdentificationFromCoimageToImageObject was added relatively recently in 6c409ef825ab8b9f9a6e72b920f761ed5840174b (2019), but grepping shows no code using it. @sebastianpos Do you recall for which application you added it?

The role of CanonicalIdentificationFromCoimageToImageObject and its dual are unclear to me. Is it the same as MorphismFromCoimageToImage, just in an Abelian category? Or in any category where images and coimages are isomorphic?

Also, I missed to rename CanonicalIdentificationFromCoimageToImageObject to CanonicalIdentificationFromCoimage*Object*ToImageObject in 81d776732406aada5f5eef845d659538abe8cd4a. But then again, MorphismFromCoimageToImage does not contain Object at all. I guess the names should be consistent.

zickgraf avatar Sep 18 '23 10:09 zickgraf

I cannot recall. I'm sorry.

It probably was meant for the abelian context. Anyway, you may get rid of it if you want to.

sebastianpos avatar Sep 18 '23 15:09 sebastianpos

@sebastianpos Thanks for the fast reply! I'm planning to automatically dualize derivations as an application of CompilerForCAP, and since the derivations around CanonicalIdentificationFromCoimageToImageObject are not symmetric with regard to dualization, I indeed would like to get rid of it.

While trying to remove it, I noticed that there actually is a derivation for it, in Toposes: https://github.com/homalg-project/Toposes/commit/3f9d8f0fac6de69ef130ed920d5880551e127498. This derivation was added right after the introduction in CAP, so that seems to be the application. @mohamed-barakat Do you need this derivation? If yes, could you also use MorphismFromCoimageToImage?

zickgraf avatar Sep 18 '23 16:09 zickgraf

@mohamed-barakat Do you need this derivation? If yes, could you also use MorphismFromCoimageToImage?

I don't know. You can delete it an we can see :)

mohamed-barakat avatar Sep 26 '23 12:09 mohamed-barakat

Resolved by https://github.com/homalg-project/CAP_project/pull/1470.

zickgraf avatar Sep 04 '24 11:09 zickgraf