return slice(0,-1) for glb_slices if glb_numb empty on an mpi rank

[deckerla]

No description provided.

[deckerla]

Closes #2003

[deckerla]

local_indices tests in MPI seem to be turned off at the moment, so I am not sure what the best way is to have unit tests for this PR

devito/tests/test_mpi.py

Line 2336 in 6e1dcc5

# TestSparseFunction().test_local_indices([(0.5, 0.5), (1.5, 2.5), (1.5, 1.5), (2.5, 1.5)], [[0.], [1.], [2.], [3.]]) # noqa

[mloubout]

tests in MPI seem to be turned off a

No this is the "main" to debug, i.e so that you can run python test_mpi.py. Pytest ignores this it runs all tests functions named test_*. So you can just use an extra case to the test here:

devito/tests/test_mpi.py

Line 364 in 6e1dcc5

@pytest.mark.parametrize('shape,expected', [

[codecov]

Codecov Report

Merging #2004 (e1e2bef) into master (6e1dcc5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #2004   +/-   ##
=======================================
  Coverage   87.85%   87.85%           
=======================================
  Files         214      214           
  Lines       36710    36713    +3     
  Branches     5550     5550           
=======================================
+ Hits        32250    32253    +3     
  Misses       3942     3942           
  Partials      518      518           

Impacted Files	Coverage Δ
devito/mpi/distributed.py	92.11% <100.00%> (ø)
tests/test_mpi.py	99.03% <100.00%> (+<0.01%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[deckerla]

@mloubout thanks for the advice! most recent push includes modified unit test covering the behavior

[mloubout]

Looks good thanks.

One minor CI comment but should be goog to go