#14684: Convert between free group elements considering generator names.
-------------------------+-------------------------------------------------
Reporter: | Owner: mmarco
mmarco | Status: needs_info
Type: | Milestone: sage-6.4
defect | Resolution:
Priority: major | Merged in:
Component: group | Reviewers:
theory | Work issues:
Keywords: free | Commit:
groups | b2d253d9feeab81f73eaab3e484c6acbe7d34144
Authors: | Stopgaps:
mmarco |
Report Upstream: N/A |
Branch: |
public/14684 |
Dependencies: |
-------------------------+-------------------------------------------------
Changes (by vdelecroix):
* status: needs_review => needs_info
* commit: => b2d253d9feeab81f73eaab3e484c6acbe7d34144
* branch: => public/14684
Comment:
Hello,
I created a branch from the ticket with `sage -dev`. The review can starts
from there.
What's the point of the following code in `_element_constructor_`?
{{{
if hasattr(P, '_freegroup_'):
if P.FreeGroup() is self:
return self.element_class(self, x.Tietze(), **kwds)
}}}
As far as I looked, there is no object in Sage with a `_freegroup_`
attribute.
Vincent
----
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=b2d253d9feeab81f73eaab3e484c6acbe7d34144
b2d253d]||{{{Trac #14684: make conversion between free groups aware of
generator names}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/14684#comment:9>
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.