Opened 12 years ago

Closed 12 years ago

#9394 closed defect (fixed)

latex representation of negative coefficients broken

Reported by: Burcin Erocal Owned by: Burcin Erocal
Priority: major Milestone: sage-4.6
Component: symbolics Keywords: pynac
Cc: Merged in: sage-4.6.alpha3
Authors: Burcin Erocal Reviewers: Karl-Dieter Crisman
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges


Reported by Mike Witt on sage-support:

sage: latex(t)
\left(2 I\right) \, \pi n x + \left(-2 I\right) \, \pi n

+ (-2 I ) looks really ugly.

Attachments (1)

trac_9394-leading_minus.patch (5.3 KB) - added by Burcin Erocal 12 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 12 years ago by Francis Clarke

See recent comments in #8938, where similar phenomena have been noted. 

Changed 12 years ago by Burcin Erocal

comment:2 Changed 12 years ago by Burcin Erocal

Authors: Burcin Erocal
Status: newneeds_review

With the new pynac package at #9901, we have:

sage: var('n')
sage: t = 2*I*n*pi*x - 2*I*n*pi
sage: latex(t)
2 i \, \pi n x - 2 i \, \pi n

attachment:trac_9394-leading_minus.patch contains the doctest changes. The fixes in the printing of rational functions (for #9834) are also included in this patch.

The pynac package includes patches for #9834, #9878, #9879, #9881, #9900 as well as this ticket. See the ticket description of #9901 for the list (and order) of patches associated to the new version.

comment:3 Changed 12 years ago by Karl-Dieter Crisman

Reviewers: Karl-Dieter Crisman
Status: needs_reviewpositive_review

With #9901, positive review. Do not merge until #9901 has positive review and is merged.

comment:4 Changed 12 years ago by Mitesh Patel

Merged in: sage-4.6.alpha3
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.