Allow a time dimension in `mask`

[rafaqz]

We should to let the mask have a time dimension like aux data.

@nicolasmerino41 I've been using your amazing CheckMaskEdges fix to OutwardsDispersal, and it works perfectly for my boundaries (the sea). But I realized I have lakes appearing over time in my land cover! Have you had similar problems?

[nicolasmerino41]

I don't think I have had this issue, although I don't really know what you mean. Do you mean that new masked cells appear forward in the simulation? That definitely hasn't happened to me :)

[rafaqz]

Yeahs my mask is changing over the simulation! I never had it either honestly, but your fix makes it very clear when it's happening

[nicolasmerino41]

That's quite weird. I definitely don't see how my fix could be causing such weird behavior.

I assume with IgnoreMaskEdges() you don't get this problem?

[rafaqz]

No sorry your fix isn't causing it!

I mean I only noticed it with your fix running because you can clearly see where the fix isn't working. Because the area is not actually masked, it's just a death zone in the dispersal kernel.

Your fix just makes it very clear that we need to add a time dimension to the mask. (So CheckMaskEdges can do its job everywhere even if the mask changes with time)

[nicolasmerino41]

Ok! I now see what you mean :)