Replies: 2 comments
-
@davemfish and I asked Jade about this (if she could think of a reason why spatial criteria might be rasterized differently) in a DM on slack, and her response was:
|
Beta Was this translation helpful? Give feedback.
-
I just took a quick look at the sample data. It seems commonplace for the footprint of a spatial criteria layer to exactly match the footprint of either a habitat or stressor. So I could imagine that rasterizing with different rules could result in a problem where, after rasterizing, a pixel on the margin might have presence of both the habitat & stressor, but would be missing the spatial criteria rating associated with that habitat-stressor pair. I'm not sure if Jade is suggesting this is a feature and not a bug, but it seems like a hacky way to accomplish the goal of conservatively estimating risk. A better approach in my opinion would be for the user to reduce the risk values in the spatially explicit layer, rather than have the model silently reduce its footprint during rasterization. If the criteria layers are rasterized differently from the habitat or stressor they represent -- when they all had identical vector geometries to begin with -- it's just a matter of time before someone is confused by that. |
Beta Was this translation helpful? Give feedback.
-
HRA is a raster-based model that uses rasterization to transform spatially explicit vectors into rasters ahead of the analysis. A user on the forums discovered a regression in the model where on certain inputs and at certain spatial resolutions you can end up with spatial criteria being 'buffered' due to the rasterization option
ALL_TOUCHED=TRUE
. By contrast, InVEST 3.9 usesALL_TOUCHED=FALSE
for spatial criteria only. For the sake of resolving that issue, #1120 / #1121 corrects that regression so behavior is consistent with the 3.9 version of HRA.But the question remains: why would we want to use 2 different methods of rasterization for spatial criteria?
If memory serves, spatial criteria were added in the 3.7.0 rebuild of HRA (which stuck around until it was replaced in 3.12.0), so we may not have had a reference for this before.
Related: #1079
This is a continuation of #1120
Beta Was this translation helpful? Give feedback.
All reactions