Opened 11 years ago
Closed 11 years ago
#10627 closed enhancement (fixed)
add options for printing matrices
Description
On ask.sagemath.org, it has been suggested that we should be able to print matrices using
0. We could also use a space instead of
0. Perhaps we could print
1 and
-1 for a (1,-1)-matrix. All of these could be implemented by passing optional arguments to the
str method for matrices, for example
.instead of
+instead of
-instead of
sage: mat = random_matrix(ZZ, 50) sage: mat.str(zero='.')
or
sage: mat.str(use_dot=True)
We need to figure out what options to allow for right now (although we can add more later) and what syntax to use for them.
Attachments (1)
Change History (13)
comment:1 Changed 11 years ago by
comment:2 Changed 11 years ago by
This ticket would be very useful to me. In my research, we deal with completing matrices, so we often represent matrices using +, -, and 0 (or ".") to represent positive, negative, or zero entries. Having an easy way to print out such a matrix would be nice.
comment:3 Changed 11 years ago by
I find things like
use_dot=True get complicated fast. As the options and possibilities increase, the keyword management becomes a morass.
I'd argue for
mat.str(zero=' ', plus_one='+', minus_one='-')
and think those three could be a good start.
How about having our cake and eating it too? One argument, and it is a dictionary, then use it for translations. Keywords, with or without a dictionary, create over-riding entries of the translation dictionary.
Then the code just checks for keys in the dictionary before setting an entry.
This would be nice for adjacency/incidence matrices of graphs.
$0.02 worth, Rob
comment:4 Changed 11 years ago by
How about a keyword that is a dictionary *or* a function? Then I could do something like:
def print_entry(x): if x>0: return '+' elif x<0: return '-' else: return '0' mat.str(print_entries=print_entry)
comment:5 Changed 11 years ago by
A patch against 4.6.1.rc0 (sorry, I haven't updated yet). I patched the generic Matrix class and the mod2 Matrix class. Are there any others that need to be changed?
comment:6 Changed 11 years ago by
Thanks!
Wow, that is a huge patch! Can you explain the whitespace changes?
comment:7 Changed 11 years ago by
Oops. I have my editor set up to delete trailing whitespace automatically and I forgot to trim the patch. I can remove all purely whitespace changes if you wish.
comment:8 Changed 11 years ago by
It would probably be better to make a separate ticket to clean up whitespace, if you want to do that.
Changed 11 years ago by
comment:9 Changed 11 years ago by
There are still a few whitespace changes, but I've removed all unnecessary hunks.
comment:10 Changed 11 years ago by
- Status changed from new to needs_review
comment:11 Changed 11 years ago by
- Reviewers set to Rob Beezer
- Status changed from needs_review to positive_review
Looks good. As mentioned, if there is additional behavior desired, that could be easily added through one of the mechanisms made available here.
Positive review.
comment:12 Changed 11 years ago by
- Merged in set to sage-4.6.2.alpha3
- Resolution set to fixed
- Status changed from positive_review to closed
Perhaps we could pass a map for how to display things like:
Then we could map things arbitrarily. It might not as easy to remember or use, but it would allow the user to do anything they wanted. I can't think when it might be useful except perhaps to make matrices of group elements more understandable or for a mathematical puzzle in which the numbers are encoded as symbols.
Actually it might be useful to show some oft repeated but complicated elements as a single letter like GAP does when displaying character tables. For that to be really useful it would be best if it were automatic, but that's probably too much to ask.