#17898: Removal of wrong stopgap
-------------------------------------+-------------------------------------
Reporter: aschilling | Owner:
Type: defect | Status: positive_review
Priority: major | Milestone: sage-6.6
Component: combinatorics | Resolution:
Keywords: stopgap, | Merged in:
partitions | Reviewers: Travis Scrimshaw,
Authors: Travis Scrimshaw, | Anne Schilling
Anne Schilling | Work issues:
Report Upstream: N/A | Commit:
Branch: | d3de7cf960cb38e03d69d3e4b8951bcc9ddd830a
public/combinat/fix_bad_stopgap-17898| Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by nbruin):
There's a problem with stopgap messages, especially when they are only
printed for some parameter choices and/or get triggered internally, via a
route that may not be easily derived by the user:
- the message only gets printed, so it may easily get lost in other
output
- the message only gets printed ''once''. The first time this happens,
the user can (at least in theory) know where they put unguaranteed
parameters in. After that, however, the protection is lost: both valid and
invalid parameters lead to the same (silent) behaviour.
So, I think the message in practice is actually less informative than one
would initially think, especially when it's printed conditionally. Extra
reason to resolve this ticket quickly and properly, and get to a situation
where the routine either returns correct answers (for a reasonable
interpretation of the input) or raises an error. Stopgaps are really a
very poor substitute for that.
--
Ticket URL: <http://trac.sagemath.org/ticket/17898#comment:35>
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 http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
