Opened 10 years ago

Last modified 7 years ago

#10787 new defect

Operation tables: multiline names for elements, latex names for elements

Reported by: jhpalmieri Owned by: joyner
Priority: minor Milestone: sage-6.4
Component: group theory Keywords:
Cc: rbeezer, nthiery, jhpalmieri, kcrisman Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges


In operation tables like Cayley tables, there are two problems right now:

  • if an element's name takes more than one line, the output looks bad. This example was reported on
    sage: G=SL(2,ZZ)
    sage: identity = matrix(ZZ, [[1,0], [0,1]])
    sage: G.cayley_table(names='elements',elements=[identity, -identity])
  • when producing LaTeX output, the LaTeX code for the element should be used rather than its _repr_. Then matrices (for example) would work well automatically.

Change History (8)

comment:1 Changed 10 years ago by jhpalmieri

  • Cc nthiery added; robert.marik removed

comment:2 Changed 10 years ago by rbeezer

  • Cc kcrisman added

Yes, lets close #10786 and work on this here.

John's answer on is a good one, give the elements simple strings as names. Using the "real" elements was meant for things like integers mod n or short permutations, the alternate names were meant for big messes (like matrices). Maybe a documentation upgrade is in order. Or we could test for multiline names (look for \n) and fail gracefully. Getting the textual output to "work" with multiline elements would be a nightmare.

So, yes, this is a job for latex (and the _repr_ fix suggested. Right now this is also mildly broken in the notebook, since jsMath does not like the array environment (IIRC). I've been waiting for the MathJax upgrade to tackle that as well.

comment:3 Changed 10 years ago by kcrisman

Hilarious. Another reason not to like Yes, do close #10786 as long as you feel like this one's not too big.

Incidentally, +1 on finding a fix for arrays in the notebook!

comment:4 Changed 10 years ago by rbeezer

See related #11362 which could probably be fixed in the same pass (perhaps once MathJax is added into the notebook).

comment:5 Changed 8 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:6 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:7 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:8 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4
Note: See TracTickets for help on using tickets.