#4761: [with patch, needs work] global cputime (inclusive some subprocesses like
Singular)
-------------------------+--------------------------------------------------
 Reporter:  malb         |        Owner:  malb      
     Type:  enhancement  |       Status:  new       
 Priority:  major        |    Milestone:  sage-3.2.2
Component:  misc         |   Resolution:            
 Keywords:               |  
-------------------------+--------------------------------------------------
Changes (by was):

  * summary:  [with patch, needs review] global cputime (inclusive some
              subprocesses like Singular) => [with patch,
              needs work] global cputime (inclusive some
              subprocesses like Singular)

Comment:

 REFEREE REPORT:
  * typo -- "be started an terminated at any given time.": The "an" should
 be "and".

  * typo -- "somehwat random": should be "somewhat random".

  * Design issues.  I think if t is output from cputime(subprocesses=True),
 then cputime(t) should automatically have subprocesses=True, since with it
 False, isn't it just giving nonsense?  Or is it really giving the Sage
 part?  If the later, ignore this comment, but please document this in the
 function.  (Yes, it's the latter.)

  * Maybe a sentence that explained that so far at least all we're timing
 is the pexpect interface subprocesses that have a working cputime method.
 That any other subprocess (e.g., gfan) isn't timed.   Then people know
 exactly what they would have to do to make sure there expect interface
 gets included.

  * The __float__ method for GlobalCputime doesn't have the examples in an
 EXAMPLES block and they aren't indented properly.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4761#comment:6>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of 
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to