#11751: make free_module_generic_pid also work for pid's other than integers
---------------------------------------------+------------------------------
Reporter: mderickx | Owner: jason, was
Type: defect | Status: positive_review
Priority: major | Milestone: sage-4.7.2
Component: linear algebra | Keywords:
Work_issues: | Upstream: N/A
Reviewer: Julian Rueth, Maarten Derickx | Author: Maarten
Derickx, Julian Rueth
Merged: | Dependencies:
---------------------------------------------+------------------------------
Changes (by leif):
* reviewer: Maarten Derickx, Julian Rueth => Julian Rueth, Maarten
Derickx
Old description:
> Before this patch the following code would fail
> {{{
> sage: R.<x>=QQ[]
> sage: L=R^1
> sage: L.span([(1/x,)])
> TypeError
> }}}
>
> Now it works
> {{{
> sage: R.<x>=QQ[]
> sage: L=R^1
> sage: L.span([(1/x,)])
> Free module of degree 1 and rank 1 over Univariate Polynomial Ring in x
> over Rational Field
> Echelon basis matrix:
> [1/x]
> }}}
>
> apply: 11751_free_module_generic_pid-fix.patch,
> trac_11751_free_module_generic_pid-review.patch and
> trac_11751_whitespace.patch
New description:
Before this patch the following code would fail
{{{
sage: R.<x>=QQ[]
sage: L=R^1
sage: L.span([(1/x,)])
TypeError
}}}
Now it works
{{{
sage: R.<x>=QQ[]
sage: L=R^1
sage: L.span([(1/x,)])
Free module of degree 1 and rank 1 over Univariate Polynomial Ring in x
over Rational Field
Echelon basis matrix:
[1/x]
}}}
----
Apply
1. [attachment:11751_free_module_generic_pid-fix.patch]
1. [attachment:trac_11751_free_module_generic_pid-review.patch]
1. [attachment:trac_11751_whitespace.patch]
to the Sage library.
--
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11751#comment:14>
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.
