Typesetting of sec(x), csc(x), cot(x) are broken

[golam-m-hossain]

Typesettings of sec(x), csc(x), cot(x) are broken. It puts an
extra "\mbox" around them. However, typesetting for sin(x),
cos(x), tan(x) works as expected.

CC: @ncalexan

Component: symbolics

Author: John Palmieri

Reviewer: Nick Alexander

Merged: 4.0.2.alpha0

Issue created by migration from https://trac.sagemath.org/ticket/6268

[sagetrac-mvngu]

comment:1

[mvngu@sage sage-4.0.1]$ sage
----------------------------------------------------------------------
| Sage Version 4.0.1, Release Date: 2009-06-06                       |
| Type notebook() for the GUI, and license() for information.        |
----------------------------------------------------------------------
sage: # the following work as expected
sage: latex(sec)
\sec
sage: latex(csc)
\csc
sage: latex(cot)
\cot
sage: # but the following have extra "\mbox" around the trig function name
sage: latex(sec(x))
\mbox{\sec}\left(x\right)
sage: latex(csc(x))
\mbox{\csc}\left(x\right)
sage: latex(cot(x))
\mbox{\cot}\left(x\right)

[jhpalmieri]

comment:2

Is this a bug in ginac/pynac? Look at this:

sage: SR
Symbolic Ring
sage: SR._latex_element_(sin(x))
'\\sin\\left(x\\right)'
sage: SR._latex_element_(sec(x))
'\\mbox{\\sec}\\left(x\\right)'

The method _latex_element_ is a one-liner:

        return GEx_to_str_latex(&x._gobj)

and I think GEx_to_str_latex is a ginac/pynac thing. At least, I found it in sage/libs/ginac/decl.pxi.

It's possible to work around it, I think, with a patch like the attached, but I'm not at all convinced that this is the right way to fix it. If you think it's okay, feel free to review it, but since I'm not sure, I've labeled it as "not read for review".

[golam-m-hossain]

comment:3

Attachment: trac_6268.patch.gz

The rebased patch for the ticket

#5711

will resolve this issue as an un-intended consequence.

[burcin]

new patch, apply only this one

[burcin]

comment:4

Attachment: trac_6268-py_print_latex.patch.gz

attachment: trac_6268-py_print_latex.patch fixes the reported problem. Apply only this patch.

[ncalexan]

Author: Burcin Erocal

[ncalexan]

Reviewer: Nick Alexander

[ncalexan]

Merged: 4.0.2.alpha0

[burcin]

Changed author from Burcin Erocal to Burcin Erocal, John Palmieri

[burcin]

Changed author from Burcin Erocal, John Palmieri to John Palmieri