#12876: Fix element and parent classes of Hom categories to be abstract, and
simplify the Hom logic.
------------------------------------------------------------------------------------------------+
Reporter: nthiery
| Owner: nthiery
Type: enhancement
| Status: needs_info
Priority: major
| Milestone: sage-5.10
Component: categories
| Resolution:
Keywords: categories, Hom
| Work issues:
Report Upstream: N/A
| Reviewers: Simon King
Authors: Nicolas M. Thiéry
| Merged in:
Dependencies: #715, #11521, #12215, #12313, #13412, #13145, #14159, #13184,
#14287, #14217 | Stopgaps:
------------------------------------------------------------------------------------------------+
Comment (by nthiery):
Replying to [comment:141 SimonKing]:
> From the commit message:
> - If category is None, Hom simply calls itself with the meet of the
> categories of the parent, which removes a cache handling duplication.
>
> I don't know if that's a good idea. Duplicating the cache costs
> virtually nothing (because all objects, categories and homsets
> involved are there anyway), but it speeds up the most commonly used
> case, namely category=None. Computing the meet of the underlying
> categories is expensive, even though it is cached.
Oh you are right. I meant to avoid a duplication in the source code, not
in the caching itself. I agree with you, the current code is wrong. I am
going to fix this right now.
Thanks for catching this!
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12876#comment:145>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
