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