Fixup _drop_halospots

[FabioLuporini]

fixes #1490

[codecov]

Codecov Report

Merging #1495 (89d7e57) into master (dc504d6) will decrease coverage by 21.88%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master    #1495       +/-   ##
===========================================
- Coverage   86.52%   64.63%   -21.89%     
===========================================
  Files         197      113       -84     
  Lines       28506    15627    -12879     
  Branches     3875     2876      -999     
===========================================
- Hits        24664    10101    -14563     
- Misses       3408     5023     +1615     
- Partials      434      503       +69     

Impacted Files	Coverage Δ
devito/passes/iet/mpi.py	64.74% <100.00%> (-34.60%)	⬇️
devito/builtins.py	11.57% <0.00%> (-79.17%)	⬇️
devito/core/autotuning.py	12.77% <0.00%> (-77.10%)	⬇️
devito/passes/clusters/aliases.py	20.75% <0.00%> (-75.61%)	⬇️
devito/mpi/routines.py	23.02% <0.00%> (-72.14%)	⬇️
devito/finite_differences/coefficients.py	18.34% <0.00%> (-70.45%)	⬇️
devito/data/utils.py	23.14% <0.00%> (-69.44%)	⬇️
devito/core/operator.py	37.93% <0.00%> (-62.07%)	⬇️
devito/data/data.py	47.01% <0.00%> (-47.43%)	⬇️
tests/conftest.py	38.65% <0.00%> (-46.22%)	⬇️
... and 158 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc504d6...ffe195a. Read the comment docs.

[mloubout]

Do you need that flag?

for d, v in dmapper.items():
    if d in hs.dimensions and all(i.is_Sequential for i in v):
          mapper[hs].update(set(hs.functions))
          break

doesn't this do the same?

[FabioLuporini]

oh yes ! I'm an idiot -- basically some leftover from some more complicated code I had before. Just running tests locally with your version. Will push the update if all remains green (as it should!)