Avoid division by zero in rechunk

[mrocklin]

cc @pitrou does this look ok to you?

[jcrist]

Perhaps:

with warnings.catch_warnings(record=True) as w:
    x = x.rechunk((N, 1, 100))
assert not w

https://docs.python.org/2/library/warnings.html#testing-warnings

[mrocklin]

Fixed

…

On Tue, Jan 10, 2017 at 9:42 AM, Jim Crist ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In dask/array/tests/test_rechunk.py <#1902 (review)>: > @@ -361,3 +361,13 @@ def test_plan_rechunk_heterogenous(): # * cf -> fc also fits the bill (graph size and block size neutral) steps = _plan((cc, ff, cf), (ff, cf, cc), block_size_limit=100) _assert_steps(steps, [(cc, ff, cc), (ff, cf, cc)]) + + +def test_rechunk_warning(capsys): + N = 20 + x = da.random.normal(size=(N, N, 100), chunks=(1, N, 100)) + x = x.rechunk((N, 1, 100)) Perhaps: with warnings.catch_warnings(record=True) as w: x = x.rechunk((N, 1, 100))assert not w https://docs.python.org/2/library/warnings.html#testing-warnings — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1902 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AASszD7T9EPnuWGpWKLQw6cqWFEjNhqeks5rQ8LogaJpZM4LfsLe> .