#12395: Change some random variable indexes that cause doctest failures if
doctests
are run in a different order.
-----------------------+----------------------------------------------------
Reporter: roed | Owner: mvngu
Type: defect | Status: needs_review
Priority: minor | Milestone: sage-5.0
Component: doctest | Keywords:
Work_issues: | Upstream: N/A
Reviewer: | Author: David Roe
Merged: | Dependencies:
-----------------------+----------------------------------------------------
Comment(by kcrisman):
Replying to [comment:3 roed]:
> The motivation for these particular changes is that Robert and I are
rewriting the doctest framework in Sage (in order to clean up the code,
use less temp files, perform timing regression testing...), and these
tests break in the new framework since code is executed in a slightly
different code path.
I see. I've heard a little about this - curious as to where the main
ticket is for this, though doubtless I wouldn't understand much :) Fewer
temp files is certainly a good thing, as is the timing issue - it always
seems to bite us.
> If you don't think the maxima ones are good changes, we can postpone
them to the patch that actually accompanies the new doctesting scripts and
just change things to the new numbers.
Probably that's better, at least for now. If we had "dummy numbers" like
`z123` it might be different, but the `z...` could look confusing to a
newbie, I think.
> Once we're agreed on the right approach I'll go back and change the
commit message.
Again, I'm surprised there aren't more of the 3d plot guys to change.
Maybe these are just the ones whose order changed.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12395#comment:4>
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.
