#16458: Braid group creates elements in wrong class
-------------------------------------+-------------------------------------
Reporter: mmarco | Owner:
Type: defect | Status: new
Priority: major | Milestone: sage-6.3
Component: group theory | Resolution:
Keywords: | Merged in:
Authors: | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/mmarco/ticket/16458 | 4e332f3e158cf0652af69827bb32701fa6565abe
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by vdelecroix):
Hello Miguel,
The `TestSuite` is here to detect this kind of things...
{{{
sage: B = BraidGroup(3)
sage: TestSuite(B).run()
}}}
The tests are run on elements that are obtained from the method
`an_element` or `some_elements`. Right now it uses a generic method which
does not rely on `_element_constructor_`. But as you can see (with the old
version)
{{{
sage: TestSuite(B([1,2])).run()
Traceback (most recent call last):
...
The following tests failed: _test_category
}}}
Please, implement in `BraidGroup_class`:
- a method `an_element` that return a braid
- a method `some_elements` that return some braids (built in different
ways)
And then put the `TestSuite` test in the `__init__`.
Vincent
--
Ticket URL: <http://trac.sagemath.org/ticket/16458#comment:3>
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.
