#9370: customize printing of elements in CombinatorialFreeModules
-------------------------------+--------------------------------------------
Reporter: jhpalmieri | Owner: sage-combinat
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.6.2
Component: combinatorics | Keywords:
Author: John Palmieri | Upstream: N/A
Reviewer: Christian Stump | Merged:
Work_issues: |
-------------------------------+--------------------------------------------
Changes (by stumpc5):
* reviewer: => Christian Stump
Comment:
Hi,
I will fold #10852 in here, if that's okay.
I like the multiple options! I gonna have a more detailed look for a
review; here are two first comments:
1. Why do you handle the old prefix option extra? Wouldn't it be enough to
keep the prefix method and return to print_options()[\'prefix\'], rather
thank keeping _prefix. A complete doctest provided by the buildbot can
then check if _prefix is used anywhere.
2. in lines 155p, multiplication is set to ast. Isn't there a way to give
this argument to the repr_lincomb method? Otherwise, we get the following:
{{{
sage: A = CombinatorialFreeModule(QQ,['+','*'],scalar_mult='@@')
sage: A.an_element()
2@@B['@@'] + 2@@B['+']
}}}
Best, Christian
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9370#comment:11>
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.
