#12005: Relativize should take one 'name' if a morphism is given
-------------------------------------+-------------------------------------
Reporter: jdemeyer | Owner: davidloeffler
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.5
Component: number fields | Resolution:
Keywords: | Merged in:
Authors: Alex Ghitza | Reviewers: Jeroen Demeyer
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/jdemeyer/ticket/12005 | 90f9d7b986913b938e4da11b402d7bd5507a5439
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by lftabera):
The code looks good, it also takes care of come hardcoded variable names.
Jeroen, do you think that taking as a new name
K.variable_name()*2
is better than, taking, for instance K.variable_name()+'0' ?
In the first case, the length of the name is doubled. I think that those
methods cannot be called so many times so this growth can de an issue, but
a warning always pop in my head with this kind of code.
What do you think?
--
Ticket URL: <http://trac.sagemath.org/ticket/12005#comment:15>
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/d/optout.
