#11186: Missing parentheses when typesetting coefficients of multivariate
polynomials over number fields
-----------------------------------+----------------------------------------
Reporter: jsrn | Owner: was
Type: defect | Status: needs_review
Priority: minor | Milestone:
Component: user interface | Keywords: latex, multivariate
polynomials, singular
Author: Johan S. R. Nielsen | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
-----------------------------------+----------------------------------------
Comment(by jsrn):
Slightly improved my patch as I discovered that negative coefficients we
typeset with a parenthesis. My fix simplifies some existing code as well.
I discovered simultaneously that in univariate polynomials, certain
coefficients will get parentheses around them, even when not needed, such
as sqrt{-1} (because it contains a '-' even though it's inside the
squareroot). As I copied some code from there, multivariate polynomials
now also have this behaviour. This broke a doctests, which I changed.
This is not optimal behaviour, but I didn't want to patch univariate
polynomials here (the patch might not be simple either -- parsing TeX,
eew). Also, I "introduced" the error in multivariate polynomials to this
for two reasons: first of all, without a solution to this sqrt-problem,
it's a side-effect of "doing the right thing most of the time"; namely in
cases such as the one in the ticket description. And this error is worse
than a superfluous parenthesis once in a while. Thirdly, I think that it's
nice, if all polynomials render more or less the same way.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11186#comment:2>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
