#8327: Implement the universal cyclotomic field, using Zumbroich basis
-----------------------------------------------------+----------------------
Reporter: nthiery | Owner:
Type: enhancement | Status:
needs_review
Priority: major | Milestone: sage-5.6
Component: number fields | Resolution:
Keywords: Cyclotomic field, Zumbroich basis | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Christian Stump, Simon King | Merged in:
Dependencies: #13765 | Stopgaps:
-----------------------------------------------------+----------------------
Comment (by nbruin):
You can "kick" the patchbot by putting `?kick` after the URL of the ticket
page at the patchbot. This has already happened and the errors keep
shifting (look at the last couple of reports).
I suspect it's due to the lazy import (that's where the error occurs). I
have no idea whether the lazy import here is done inappropriately, whether
there's a subtle bug in lazy import that gets triggered here, or whether
the patchbot has troubles that just happen to be triggered by this patch
(sounds unlikely).
Since two of these three can be avoided by not doing a lazy import, that's
probably the way to go if you want the ticket merged ASAP. Then
"lazification" can be done on a separate ticket. That or someone referees
the ticket with "works for me" and puts on a positive review, ignoring the
bot.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8327#comment:174>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
