-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow a time dimension in mask
#264
Comments
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 :) |
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 |
That's quite weird. I definitely don't see how my fix could be causing such weird behavior. I assume with |
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) |
Ok! I now see what you mean :) |
We should to let the mask have a time dimension like aux data.
@nicolasmerino41 I've been using your amazing
CheckMaskEdges
fix toOutwardsDispersal
, 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?The text was updated successfully, but these errors were encountered: