#5457: Refactor symmetric functions and k-bounded subspace
----------------------------------------------------------------+-----------
Reporter: nthiery |
Owner: mhansen
Type: enhancement |
Status: closed
Priority: major |
Milestone: sage-5.4
Component: combinatorics |
Resolution: fixed
Keywords: symmetric functions, days38, sd40 | Work
issues:
Report Upstream: N/A |
Reviewers: Dan Bump, Nicolas M. Thiéry, Jeroen Demeyer
Authors: Mike Zabrocki, Anne Schilling, Jason Bandlow | Merged
in: sage-5.4.beta0
Dependencies: #11563, #13109, #12969 |
Stopgaps:
----------------------------------------------------------------+-----------
Comment (by jdemeyer):
Replying to [comment:99 aschilling]:
> That's why we added this (since this can potentially be a common user
mistake). If this should be done differently, feel free to change it!
Yes, it should be done differently. The correct way would be:
{{{
if not needed_condition:
raise ValueError("Error message")
}}}
As I said: `assert` is only meant to catch ''bugs'', not user errors.
Although, as Nicolas Thiéry argued, something `assert` is acceptible. See
also the sage-devel thread.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5457#comment:100>
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.
