Changes between Version 1 and Version 7 of Ticket #26433


Ignore:
Timestamp:
10/10/18 00:56:54 (2 years ago)
Author:
tscrim
Comment:

I made a few Cython improvements. I also pulled out the if statement because I felt the code duplication was minimal and it would be better to avoid doing the if check every time in the loop.

If my changes are good, then positive review (with the author name is filled in).

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #26433

    • Property Status changed from new to needs_review
    • Property Commit changed from to 1ca2c054271ae4dfa6719efe6bae1fc6bcc331fe
    • Property Branch changed from to u/tscrim/slowness_generic_quo_rem-26433
    • Property Reviewers changed from to Travis Scrimshaw
  • Ticket #26433 – Description

    v1 v7  
    101010000 loops, best of 3: 145 µs per loop
    1111
    12 sage: Q = X - t
     12sage: Q = x - t
    1313sage: %timeit _ = P % Q
    14141000 loops, best of 3: 1.06 ms per loop