#20094: Fix and clean up xsrange
-------------------------------------+-------------------------------------
Reporter: dkrenn | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-7.1
Component: misc | Resolution:
Keywords: | Merged in:
Authors: Jeroen Demeyer | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/jdemeyer/xsrange_fails_with_large_inputs|
2f7f7582ca3ffcbd74c46dec105aaf57226ee0c7
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by jdemeyer):
* status: new => needs_review
* commit: => 2f7f7582ca3ffcbd74c46dec105aaf57226ee0c7
Old description:
> We have
> {{{
> sage: next(xsrange(0, 2^63-2^10+2^9))
> ...
> OverflowError: Python int too large to convert to C long
> }}}
> This is because the internally used variable `count` is too large.
>
> We solve this problem. While we're at it, we also move `xsrange()` and
> related functions to a new Cython module `src/sage/arith/srange.pyx`.
New description:
We have
{{{
sage: next(xsrange(0, 2^63-2^10+2^9))
...
OverflowError: Python int too large to convert to C long
}}}
This is because the internally used variable `count` is too large.
We solve this problem by avoiding `xrange()`. While we're at it, we also
move `xsrange()` and related functions to a new Cython module
`src/sage/arith/srange.pyx`.
--
Comment:
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=2f7f7582ca3ffcbd74c46dec105aaf57226ee0c7
2f7f758]||{{{Implement xsrange() in Cython and move to
sage.arith.srange}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/20094#comment:5>
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.
