#8150: various fixes in sage/groups/ and sage/interfaces needed for GAP 4.4.12
------------------------------+---------------------------------------------
Reporter: dimpase | Owner: joyner
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.3.2
Component: group_theory | Keywords:
Author: Dima Pasechnik | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
------------------------------+---------------------------------------------
Changes (by dimpase):
* status: new => needs_review
Comment:
Replying to [comment:1 robertwb]:
> {{{# random order}}} is preferable to {{{#not tested}}}.
I can change this, no problem.
> I think a lot of these could be tested in sensible ways. For example, if
word_problem returned a list of *tuples* rather than lists (which is more
natural), then one could do
>
{{{
sage: w = word_problem([a*b,a*c], b*c)
sage: set(w) == set([(a*b, 1), (a*c, 1)])
}}}
This would mean a redesign. It is doable, I suppose, but I do not know the
reason for the original decision to have a list of lists. And I don't see
this as urgent, as this would give, at best, a better readable code.
>
> Even better,
>
{{{
sage: w = word_problem([a*b,a*c], b*c); w # random solution
[[a*b, 1], [a*c, 1]]
sage: w == prod(g^e for g,e in w)
True
}}}
>
> which demonstrates that w is indeed a solution.
well, this is not working, at least not presently:
{{{
d...@boxen:~/sage$ ./sage
----------------------------------------------------------------------
| Sage Version 4.3.2.alpha1, Release Date: 2010-01-31 |
| Type notebook() for the GUI, and license() for information. |
----------------------------------------------------------------------
**********************************************************************
* *
* Warning: this is a prerelease version, and it may be unstable. *
* *
**********************************************************************
sage: G.<a,b,c> = AbelianGroup(3,[2,3,4])
sage: w = word_problem([a*b,a*c], b*c)
sage: w == prod(g^e for g,e in w)
False
sage: gap_version()
'4.4.12'
}}}
>
> Also, it's clearer to us any and all instead of reduce, if you need to.
Having huge blocks of {{{#not tested}}} should be avoided unless there's
no clean way around it.
There is no clean way around them without a major redesign.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8150#comment:2>
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.
