#11461: make @parallel work with class/instance methods
-----------------------------------+----------------------------------------
   Reporter:  niles                |          Owner:  tbd                    
       Type:  defect               |         Status:  needs_review           
   Priority:  major                |      Milestone:  sage-4.7.1             
  Component:  performance          |       Keywords:  parallel decorator sd31
Work_issues:                       |       Upstream:  N/A                    
   Reviewer:  Karl-Dieter Crisman  |         Author:  Mike Hansen            
     Merged:                       |   Dependencies:                         
-----------------------------------+----------------------------------------
Changes (by kcrisman):

 * cc: niles (added)
  * reviewer:  => Karl-Dieter Crisman


Comment:

 Formatting things:
  * This needs very minor rebasing with respect to #11467.
  * It should also have at least one example in the !ParallelFunction class
 definition (there should be a docstring) so that something shows up in the
 reference manual.
  * The end of the ``__call__`` docstring seems to have the triple quote
 misindented.
  * `except AtrributeError:` this isn't really a formatting error, but just
 an error.  There should be a doctest for this?
  * `anqd` is not a word

 It looks good, though.  I don't feel quite comfortable enough with the
 parallel stuff to review it completely, but if the doctests work then this
 makes sense, and the getdoc etc. certainly is right.

 I'm cc:ing Niles in because he originally asked about this, so hopefully
 he can positively review the rest.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11461#comment:7>
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.

Reply via email to