#18029: speed up integral point enumeration
-------------------------------------+-------------------------------------
Reporter: vdelecroix | Owner:
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-7.3
Component: geometry | Resolution:
Keywords: | Merged in:
Authors: Travis Scrimshaw | Reviewers: Matthias Koeppe
Report Upstream: N/A | Work issues:
Branch: | Commit:
public/geometry/speedup_integral_points-18029|
801226044e890ac38db3bbe6367cd4692b3095d8
Dependencies: #17562 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by jdemeyer):
You need to use an `except` value when you have a `c(p)def` function with
a return type: replace `cpdef bint ...(...)` by `cpdef bint ...(...)
except -1`.
And since you seem to like Cython micro-optimizations a lot in this
ticket, you missed an obvious one: setting the `boundscheck` and
`wraparound` [http://docs.cython.org/src/reference/compilation.html
#compiler-directives directives] to `False` to speed up list indexing.
--
Ticket URL: <https://trac.sagemath.org/ticket/18029#comment:17>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
