#18100: Parallelization of computations on tensors on free modules
-------------------------------------+-------------------------------------
       Reporter:  egourgoulhon       |        Owner:
           Type:  enhancement        |       Status:  needs_info
       Priority:  major              |    Milestone:  sage-6.8
      Component:  linear algebra     |   Resolution:
       Keywords:  parallelization,   |    Merged in:
  free module, tensor                |    Reviewers:
        Authors:  Marco Mancini      |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  ea38c84d685446a1de5793cd00d79ea6c5448605
  public/18100-parallelization_tensors|     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by mmancini):

 Replying to [comment:13 vdelecroix]:
 Hello Vincent,

 Thanks to review our work.

 We try to reply to the two points:
 1) You are right, the names `set_nproc` and `get_nproc` are generic names
 and not clearly referred to tensors, so we propose to changes them in
 `set_nproc_tensor` and `get_nproc_tensor`.

 2) The functions `set_nproc` and `get_nproc`  were introduced in order to
 avoid
 to the users to typing the two more verbose (and less clear) functions
 `TensorParallelCompute().set(nproc)` and  `TensorParallelCompute._nproc`
 Note that the singleton `TensorParallelCompute` is not in the global
 namespace but only the functions `set_nproc` and `get_nproc`.

 Best regards,

 Marco

 > Hello,
 >
 > The name `set_nproc` is not appropriate for a function that is intended
 to be used only for tensor products. Secondly, why are you duplicating the
 interface with a class `TensorParallelCompute` and functions `set_nproc`,
 `get_nproc`?
 >
 > Vincent

--
Ticket URL: <http://trac.sagemath.org/ticket/18100#comment:14>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to