id,summary,reporter,owner,description,type,status,priority,milestone,component,resolution,keywords,cc,merged,author,reviewer,upstream,work_issues,branch,commit,dependencies,stopgaps
4983,replace subdivisions attribute for matrices with a function,was,was,"I do not like this:
{{{
sage: sage: a = matrix(ZZ,4,[1, 0, 0, 0, 0, 1, 0, 0, 1, -1, 1, 0, 1, -1, 1, 2])
sage: sage: b=a.jordan_form()
sage: b.subdivisions
([0, 1, 3, 4], [0, 1, 3, 4])
sage: b.subdivisions = 10
sage: b.subdivisions
10
}}}
Notice that you can make the subdivisions nonsense because it can be changed.
Also, of course,
{{{
sage: b.subdivisions?
... The Integer class represents arbitrary precision
integers. It derives from the Element class, so
[other useless stuff]
}}}
I don't like that at all either. I wish that subdivisions were a method with a proper docstring, doctests, etc., and that variable were hidden.
Then one would do:
{{{
sage: b.subdivisions?
useful stuff (and also it would be in the reference manual)
and
sage: b.subdivisions()
([0, 1, 3, 4], [0, 1, 3, 4])
}}}
------
'''Depends on:'''
1. #10974
'''Apply:'''
1. [attachment:trac_4983-subdivisions-rebased.patch]",enhancement,closed,minor,sage-4.7,linear algebra,fixed,,,sage-4.7.alpha4,John Palmieri,Rob Beezer,N/A,,,,,