#15920: Parallelize Possible Periods functions for Projective Morphisms
-------------------------------------+-------------------------------------
Reporter: drose | Owner: drose
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-6.2
Component: algebraic | Resolution:
geometry | Merged in:
Keywords: Projective, | Reviewers: Ben Hutz
Morphism, Parallelization | Work issues:
Authors: Dillon Rose | Commit:
Report Upstream: N/A | 69dd970e65ad635573be4cc0cb0d8def1d442991
Branch: | Stopgaps:
u/drose/ticket/15920 |
Dependencies: 15966 |
-------------------------------------+-------------------------------------
Changes (by bhutz):
* status: new => needs_review
* dependencies: => 15966
Comment:
These two changes looks fine. However, there is a third place that should
be done in parallel.
{{{periodic_points=self.lift_to_rational_periodic(pos_points,B)}}}
in self.rational_periodic_points()
Also, it is convention not to use CamelCase variables names. (i.e.
parallelData, possiblePeriods should be not have the capital in them).
I'm also marking 15966 as a dependency since without the cleanup of the
child process these create too many child process to function well.
--
Ticket URL: <http://trac.sagemath.org/ticket/15920#comment:9>
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.
