#15820: Implement sequences of bounded integers
-------------------------------------+-------------------------------------
       Reporter:  SimonKing          |        Owner:
           Type:  enhancement        |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.3
      Component:  algebra            |   Resolution:
       Keywords:  sequence bounded   |    Merged in:
  integer                            |    Reviewers:
        Authors:  Simon King         |  Work issues:  Fix flakyness in
Report Upstream:  N/A                |  adding sequences
         Branch:                     |       Commit:
  u/SimonKing/ticket/15820           |  735939e593c9c302ff42c4973cbfe2e48eb22644
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by SimonKing):

 * status:  needs_review => needs_work
 * work_issues:  Use mpn_* for speedup of iteration and item access => Fix
     flakyness in adding sequences


Comment:

 I found failures in ''some'' runs:
 {{{
 king@linux-etl7:~/Sage/git/sage> ./sage -t
 src/sage/misc/bounded_integer_sequences.pyx
 Running doctests with ID 2014-06-03-13-33-10-7f6bcfa0.
 Doctesting 1 file.
 sage -t src/sage/misc/bounded_integer_sequences.pyx
 **********************************************************************
 File "src/sage/misc/bounded_integer_sequences.pyx", line 1059, in
 sage.misc.bounded_integer_sequences.BoundedIntegerSequence.__add__
 Failed example:
     S+T
 Expected:
     <4, 1, 6, 2, 7, 20, 9, 4, 1, 6, 2, 8, 15>
 Got:
     <4, 1, 6, 2, 7, 20, 9, 4, 1, 6, 2, 8, 31>
 **********************************************************************
 1 item had failures:
    1 of  10 in
 sage.misc.bounded_integer_sequences.BoundedIntegerSequence.__add__
     [178 tests, 1 failure, 0.79 s]
 ----------------------------------------------------------------------
 sage -t src/sage/misc/bounded_integer_sequences.pyx  # 1 doctest failed
 ----------------------------------------------------------------------
 Total time for all tests: 0.9 seconds
     cpu time: 0.8 seconds
     cumulative wall time: 0.8 seconds
 king@linux-etl7:~/Sage/git/sage> ./sage -t
 src/sage/misc/bounded_integer_sequences.pyx
 Running doctests with ID 2014-06-03-13-33-15-6cf37501.
 Doctesting 1 file.
 sage -t src/sage/misc/bounded_integer_sequences.pyx
     [178 tests, 0.78 s]
 ----------------------------------------------------------------------
 All tests passed!
 ----------------------------------------------------------------------
 Total time for all tests: 0.9 seconds
     cpu time: 0.8 seconds
     cumulative wall time: 0.8 seconds
 king@linux-etl7:~/Sage/git/sage> ./sage -t
 src/sage/misc/bounded_integer_sequences.pyx
 Running doctests with ID 2014-06-03-13-33-20-d43d6691.
 Doctesting 1 file.
 sage -t src/sage/misc/bounded_integer_sequences.pyx
 **********************************************************************
 File "src/sage/misc/bounded_integer_sequences.pyx", line 1061, in
 sage.misc.bounded_integer_sequences.BoundedIntegerSequence.__add__
 Failed example:
     T+S
 Expected:
     <4, 1, 6, 2, 8, 15, 4, 1, 6, 2, 7, 20, 9>
 Got:
     <4, 1, 6, 2, 8, 15, 4, 1, 6, 2, 7, 20, 25>
 **********************************************************************
 1 item had failures:
    1 of  10 in
 sage.misc.bounded_integer_sequences.BoundedIntegerSequence.__add__
     [178 tests, 1 failure, 0.78 s]
 ----------------------------------------------------------------------
 sage -t src/sage/misc/bounded_integer_sequences.pyx  # 1 doctest failed
 ----------------------------------------------------------------------
 Total time for all tests: 0.9 seconds
     cpu time: 0.8 seconds
     cumulative wall time: 0.8 seconds
 king@linux-etl7:~/Sage/git/sage> ./sage -t
 src/sage/misc/bounded_integer_sequences.pyx
 Running doctests with ID 2014-06-03-13-33-25-04f71350.
 Doctesting 1 file.
 sage -t src/sage/misc/bounded_integer_sequences.pyx
     [178 tests, 0.78 s]
 ----------------------------------------------------------------------
 All tests passed!
 ----------------------------------------------------------------------
 Total time for all tests: 0.9 seconds
     cpu time: 0.8 seconds
     cumulative wall time: 0.8 seconds
 king@linux-etl7:~/Sage/git/sage> ./sage -t
 src/sage/misc/bounded_integer_sequences.pyx
 Running doctests with ID 2014-06-03-13-33-29-a2a81517.
 Doctesting 1 file.
 sage -t src/sage/misc/bounded_integer_sequences.pyx
     [178 tests, 0.79 s]
 ----------------------------------------------------------------------
 All tests passed!
 ----------------------------------------------------------------------
 Total time for all tests: 0.9 seconds
     cpu time: 0.8 seconds
     cumulative wall time: 0.8 seconds
 king@linux-etl7:~/Sage/git/sage> ./sage -t
 src/sage/misc/bounded_integer_sequences.pyx
 Running doctests with ID 2014-06-03-13-33-33-e89cc5fe.
 Doctesting 1 file.
 sage -t src/sage/misc/bounded_integer_sequences.pyx
     [178 tests, 0.81 s]
 ----------------------------------------------------------------------
 All tests passed!
 ----------------------------------------------------------------------
 Total time for all tests: 0.9 seconds
     cpu time: 0.8 seconds
     cumulative wall time: 0.8 seconds
 king@linux-etl7:~/Sage/git/sage> ./sage -t
 src/sage/misc/bounded_integer_sequences.pyx
 Running doctests with ID 2014-06-03-13-33-38-83426cf0.
 Doctesting 1 file.
 sage -t src/sage/misc/bounded_integer_sequences.pyx
     [178 tests, 0.79 s]
 ----------------------------------------------------------------------
 All tests passed!
 ----------------------------------------------------------------------
 Total time for all tests: 0.9 seconds
     cpu time: 0.8 seconds
     cumulative wall time: 0.8 seconds
 }}}

 My first guess is that this comes from using `sage_malloc` in some places,
 i.e., I should zero the memory first.

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