#15110: Bug in matrix constructor
-------------------------------------+-------------------------------------
Reporter: caruso | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.4
Component: linear algebra | Resolution:
Keywords: matrix | Merged in:
constructor | Reviewers:
Authors: Vincent Delecroix | Work issues:
Report Upstream: N/A | Commit:
Branch: | fff34482450a430a8c58da4d5492f8d94b8a381b
u/vdelecroix/15110 | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Changes (by {'newvalue': u'Vincent Delecroix', 'oldvalue': u'Xavier Caruso'}):
* status: needs_work => needs_review
* commit: => fff34482450a430a8c58da4d5492f8d94b8a381b
* branch: => u/vdelecroix/15110
* author: Xavier Caruso => Vincent Delecroix
Old description:
> Here is the bug
>
> {{{
> sage: S.<t> = LaurentSeriesRing(ZZ)
> sage: MS = MatrixSpace(S,1,1)
> sage: MS([t])
> [1]
> }}}
>
> (The reason is the following: Sage makes a confusion between `t` and its
> list of coefficients and hence do as there was written `MS([[1]])`.)
>
> The attached patch fixes this. I hope that it breaks nothing else.
New description:
Here is the bug
{{{
sage: S.<t> = LaurentSeriesRing(ZZ)
sage: MS = MatrixSpace(S,1,1)
sage: MS([t])
[1]
}}}
It is correct in sage-6.6.rc2, the associated branch just adds a doctest.
--
Comment:
Last 10 new commits:
||[http://git.sagemath.org/sage.git/commit/?id=23eb30ccad2bb47f20e22810176733660d3c6a85
23eb30c]||{{{Trac 18152: implementation of UCF using libgap}}}||
||[http://git.sagemath.org/sage.git/commit/?id=99c8d80c5f32090bbc63cfbdc63695602975f477
99c8d80]||{{{Trac 18152: remove the old implementation}}}||
||[http://git.sagemath.org/sage.git/commit/?id=7d957347ccfeaba97438c92f6c04cbf79180f0fc
7d95734]||{{{Trac 18152: Fix coxeter group import}}}||
||[http://git.sagemath.org/sage.git/commit/?id=36fb6e5fdf3bb7879e7b67c8c40cf69a045ac144
36fb6e5]||{{{Trac 18153: better __int__ and add support for -infinity}}}||
||[http://git.sagemath.org/sage.git/commit/?id=e34e9ca6da48db87eb7ec594e6cc56e940a9aff2
e34e9ca]||{{{Trac 18152: fix documentation}}}||
||[http://git.sagemath.org/sage.git/commit/?id=376bcc2dd953ebe96ed5594d99ce4155ccbd883c
376bcc2]||{{{Trac 18152: fix constructor in
rings.number_field.number_field}}}||
||[http://git.sagemath.org/sage.git/commit/?id=e96846846f65b0a914d276f34725f97bfd974f4a
e968468]||{{{Trac 18152: better cmp + 32 vs 64 bits in doctest}}}||
||[http://git.sagemath.org/sage.git/commit/?id=e58f75501bdd8dfd4b20bc751a6040e5b10f0844
e58f755]||{{{Trac 18152: add a _sub_ method}}}||
||[http://git.sagemath.org/sage.git/commit/?id=e4a352468cf9c018b9bdf148a3de03793a111b9e
e4a3524]||{{{Trac 18152: speedup (use X._add_(Y) instead of X+Y)}}}||
||[http://git.sagemath.org/sage.git/commit/?id=fff34482450a430a8c58da4d5492f8d94b8a381b
fff3448]||{{{Trac 15110: add a doctest to MatrixSpace.__call__}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/15110#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.
