Opened 12 years ago

Closed 12 years ago

#9017 closed enhancement (fixed)

Use lowercase i for latex symbolic sqrt(-1)

Reported by: Robert Bradshaw Owned by: Burcin Erocal
Priority: major Milestone: sage-4.4.4
Component: calculus Keywords:
Cc: Merged in: sage-4.4.4.alpha0
Authors: Robert Bradshaw Reviewers: Burcin Erocal
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description


Attachments (3)

9017-latex-sqrt-neg1.patch (3.9 KB) - added by Robert Bradshaw 12 years ago.
9017-fixes.patch (1.3 KB) - added by Robert Bradshaw 12 years ago.
Here's one fix.
trac_9017-doctest_fixes.patch (1.0 KB) - added by Burcin Erocal 12 years ago.

Download all attachments as: .zip

Change History (9)

Changed 12 years ago by Robert Bradshaw

Attachment: 9017-latex-sqrt-neg1.patch added

comment:1 Changed 12 years ago by Robert Bradshaw

Status: newneeds_review

comment:2 Changed 12 years ago by Burcin Erocal

Authors: Robert Bradshaw
Reviewers: Burcin Erocal
Status: needs_reviewneeds_work

Wow! Thanks for the quick patch.

There are a few minor looking doctest problems. I'll post a review patch and change to positive review soon.

Changed 12 years ago by Robert Bradshaw

Attachment: 9017-fixes.patch added

Here's one fix.

comment:3 Changed 12 years ago by Robert Bradshaw

Status: needs_workneeds_review

See also #9018.

Changed 12 years ago by Burcin Erocal

comment:4 Changed 12 years ago by Burcin Erocal

Status: needs_reviewpositive_review

Surprisingly not much else is needed. attachment:trac_9017-doctest_fixes.patch fixes two doctests.

All patches on this ticket should be applied. In this order:

comment:5 Changed 12 years ago by Robert Bradshaw

Type: defectenhancement

comment:6 Changed 12 years ago by Mike Hansen

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