#2624: [with patch, needs review] parallel testing: sage -tp foo/bar/file.py
should assume 1 thread
---------------------------+------------------------------------------------
Reporter: mabshoff | Owner: jhpalmieri
Type: enhancement | Status: assigned
Priority: major | Milestone: sage-4.1.2
Component: doctest | Resolution:
Keywords: | Reviewer:
Author: John Palmieri | Merged:
---------------------------+------------------------------------------------
Comment(by ddrake):
Replying to [comment:8 jhpalmieri]:
> > With this patch, when I do something like "sage -tp fjfjfjfjfj", it
doesn't tell me that I gave it a bad file:
>
> I feel as though this belongs in another ticket. For what it's worth,
previous authors of the file seem to have made this decision
intentionally:
> {{{
> else:
> continue # prefer silence to: raise TypeError,
"Unknown File %s" % F
> }}}
Okay, that does look like another ticket.
> > Also, when it makes an assumption about the number of threads, I'd
like it to print that it's using 1 thread. (Mostly for my own
reassurance.)
>
> That's easy enough to change; see the new patch. This version also sets
numthreads to be no more than the number of files.
Those look like good changes. Magic 8-ball says...positive review likely.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/2624#comment:9>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
