#10963: More functorial constructions
-------------------------------------+-------------------------------------
Reporter: nthiery | Owner: stumpc5
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.1
Component: categories | Resolution:
Keywords: days54 | Merged in:
Authors: Nicolas M. Thiéry | Reviewers: Simon King, Frédéric
Report Upstream: N/A | Chapoton
Branch: | Work issues:
public/ticket/10963 | Commit:
Dependencies: #11224, #8327, | dbb17b11bb9e8f94b5d9d3424cd34c5efc82564c
#10193, #12895, #14516, #14722, | Stopgaps:
#13589, #14471, #15069, #15094, |
#11688, #13394, #15150, #15506 |
-------------------------------------+-------------------------------------
Comment (by nthiery):
Replying to [comment:332 SimonKing]:
> +1. I think it is important that the consistency tests take place, and I
think they guarantee robustness of the code.
>
> I can't check the code right now; but didn't one of your last commits
remove the "assert" statement that has originally triggered the recursion?
I think after my commit, the assertion would not trigger a recursion, and
it would perhaps be better to keep it in---unless you can point out that
an equivalent consistency check is happening anyway.
My commit removed the assertion enforcing that the axiom category
class itself had to be in a standard location. On the other hand, the
assertion about the consistency of links is still there (line 262).
Cheers,
Nicolas
--
Ticket URL: <http://trac.sagemath.org/ticket/10963#comment:338>
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.
For more options, visit https://groups.google.com/groups/opt_out.
