#15348: "R.<a> =" syntactic sugar incorrect for EquationOrder and ZZ.extension
-------------------------------------+-------------------------------------
Reporter: emassop | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-7.0
Component: number fields | Resolution:
Keywords: | Merged in:
Authors: Jeroen Demeyer | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/jdemeyer/ticket/15348 | e857c9a484132348092e21cd1de0711686b8b7f0
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by jdemeyer):
Replying to [comment:19 emassop]:
> Perhaps there could be `defining_generators.set(...)` that would
> * populate the cache if it has no value,
> * do nothing if the cache and the argument agree,
> * populate the cache with an invalid value when the argument and the
cache disagree.
> The invalid value would make a call to `defining_generators` raise an
exception.
I see no reason for this additional layer of complexity. I don't know of
any cached function in Sage which does this and I don't see why
`defining_generators` would need it.
> Also, how does the cache behaving with pickling? Does a cache override
get saved?
Yes, it gets pickled although this might be changed in #15692.
> Finally, does it make sense to define `defining_generators` in
`category_object.pyx` as simply `gens()`, and put the override to
`ring_generators` in `class Order`?
I don't really see why. Currently, `Order` is the only class with
`ring_generators` but if a new class comes around implementing
`ring_generators`, I guess (but this is really guessing) that it would
make sense to use it as defining generators.
--
Ticket URL: <http://trac.sagemath.org/ticket/15348#comment:20>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
