Opened 9 years ago

Closed 9 years ago

#9394 closed defect (fixed)

latex representation of negative coefficients broken

Reported by: burcin Owned by: burcin
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:

Description

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 9 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 9 years ago by fwclarke

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

Changed 9 years ago by burcin

comment:2 Changed 9 years ago by burcin

  • Authors set to Burcin Erocal
  • Status changed from new to needs_review

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

sage: var('n')
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 9 years ago by kcrisman

  • Reviewers set to Karl-Dieter Crisman
  • Status changed from needs_review to positive_review

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

comment:4 Changed 9 years ago by mpatel

  • Merged in set to sage-4.6.alpha3
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.