#17792: Word problem for FareySymbol
-------------------------------------+-------------------------------------
Reporter: mmasdeu | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.7
Component: modular forms | Resolution:
Keywords: farey symbol, | Merged in:
SL2Z, word problem | Reviewers:
Authors: Marc Masdeu | Work issues:
Report Upstream: N/A | Commit:
Branch: u/mmasdeu/17792 | 908375633e44d67fa3991bf0f7c4e2cc381dbef1
-farey-wordproblem | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by vdelecroix):
Hello,
Replying to [comment:21 mmasdeu]:
> 1) In the timings I had, it look like it was faster to create the tuple
out of the list instead of the generator expression. But I have changed it
as you proposed.
Looks strange! I do not understand why {{{[i for i in l]}}} should be
faster than {{{tuple(i for i in l)}}}...
> 2) I have reverted back to one single function as you proposed. The
output is as both John and myself had decided: the exponents can be any
nonzero integer, not just +1 or -1. I have applications for this kind of
output and so may have other potential users.
Thanks.
> 3) I use now cached_method instead manual caching. I changed the
try/except blocks with a more elegant solution (I think it is better than
what you suggest --which also doesn't work because matrices in SL2Z do not
have a unary "-"--, since it uses half the queries in a dictionary).
Nice. Do you know why SL2Z matrices do not have a unary `-`?
Vincent
--
Ticket URL: <http://trac.sagemath.org/ticket/17792#comment:22>
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.