#19500: PolynomialRing: Strange errors with invalid term order
-------------------------------------+-------------------------------------
Reporter: jdemeyer | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.10
Component: basic arithmetic | Resolution:
Keywords: | Merged in:
Authors: Jeroen Demeyer | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/jdemeyer/polynomialring__strange_errors_with_invalid_term_order|
c1b968ed57d5ebd85cc2b7c3bbfa3ff5f1159091
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by bruno):
This looks good. My sole comment is whether one could also have an
improvement for the following error message:
{{{#!python
sage: T = TermOrder("deglex(3),lex(2)")
sage: R = PolynomialRing(QQ, 4, 'x', order=T)
Traceback (most recent call last):
...
ValueError: TermOrder copy constructor called with different number of
variables (4) than input term order (5).
}}}
It would be more meaningful to get an error message along the lines of
{{{
ValueError: the length of the term order (5) is different from the number
of variables (5).
}}}
--
Ticket URL: <http://trac.sagemath.org/ticket/19500#comment:5>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
