#10925: Better error messages for bad arrow plots
-------------------------------------+-------------------------------------
Reporter: kcrisman | Owner: jason, was
Type: enhancement | Status: needs_review
Priority: minor | Milestone:
Component: graphics | Resolution:
Keywords: beginner | Merged in:
Authors: | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/philwilt/ticket/10925 | b271d8593be4d43f9ac200bed97ce59117102368
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by kcrisman):
This seems really complicated. Would it be easier to just do
{{{
try:
if len(headpoint) != len(tail point):
stuff
elif
except TypeError # kind of error when you ask for len of something that
doesn't have it
make into lists like you do
}}}
I get what this code is doing but I'm afraid it's going to be hard to
maintain. I don't know how much of an efficiency gain your solution might
be, though, or maybe there are too many cases anyway no matter how you do
it.
What do you think? By the way, it needs doctests of the "bad" cases
(there should be examples of traceback doctesting in nearby code, I think,
for sure elsewhere in the plot modules). Thanks for working on this,
it's close!
--
Ticket URL: <http://trac.sagemath.org/ticket/10925#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 http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
