Skip to content
This repository has been archived by the owner on Jan 30, 2023. It is now read-only.

Commit

Permalink
This test was failing on my Windows machine, due to the calculation f…
Browse files Browse the repository at this point in the history
…inishing before the timeout.

This is possibly due in part to the abort timer being a bit 'slower' to go off on Cygwin due to Cygwin overheads.
Changing this to a much larger calculation pretty much guarantees the test to pass though--just need to make
absolute sure the timeout works ''correctly'' :)
  • Loading branch information
embray committed Aug 12, 2016
1 parent c0ba4a7 commit b6caede
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/sage/parallel/map_reduce.py
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,7 @@
Here is an example or how to deal with timeout::
sage: from sage.parallel.map_reduce import RESetMPExample, AbortError
sage: EX = RESetMPExample(maxl = 8)
sage: EX = RESetMPExample(maxl = 100)
sage: try:
....: res = EX.run(timeout=0.01)
....: except AbortError:
Expand All @@ -225,6 +225,7 @@
The following should not timeout even on a very slow machine::
sage: EX = RESetMPExample(maxl = 8)
sage: try:
....: res = EX.run(timeout=60)
....: except AbortError:
Expand Down Expand Up @@ -1379,6 +1380,7 @@ def run(self,
Here is an example or how to deal with timeout::
sage: from sage.parallel.map_reduce import AbortError
sage: EX = RESetMPExample(maxl = 100)
sage: try:
....: res = EX.run(timeout=0.01)
....: except AbortError:
Expand All @@ -1391,6 +1393,7 @@ def run(self,
The following should not timeout even on a very slow machine::
sage: from sage.parallel.map_reduce import AbortError
sage: EX = RESetMPExample(maxl = 8)
sage: try:
....: res = EX.run(timeout=60)
....: except AbortError:
Expand Down

0 comments on commit b6caede

Please sign in to comment.