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

  * reviewer:  Karl-Dieter Crisman => Karl-Dieter Crisman, Niles Johnson
  * milestone:  sage-4.8 => sage-5.0


Old description:

> The `@parallel` decorator currently cannot be used with methods that
> implicitly prepend their arguments with a class instance, because
> `@parallel` requires that the first argument be a list (or iterable) of
> arguments to pass to the function.
>
> More details can be found at this
> [http://ask.sagemath.org/question/585/can-i-use-parallel-for-
> classinstance-methods asksage question].

New description:

 The `@parallel` decorator currently cannot be used with methods that
 implicitly prepend their arguments with a class instance, because
 `@parallel` requires that the first argument be a list (or iterable) of
 arguments to pass to the function.

 More details can be found at this
 [http://ask.sagemath.org/question/585/can-i-use-parallel-for-
 classinstance-methods asksage question].

 ----
 Apply [attachment:trac_11461_rebase_2.patch].

--

Comment:

 See also #11462.  My guess is that that has priority due to the prior
 positive review...

 Niles, my sense is that you only rebased, but give positive review to
 Mike's patch.  What would still have to be reviewed?

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