Dealing with padding

[roughhawkbit]

Potential inconsistency between SpatialGrid.isValid(DiscreteVector dC) (1) and SpatialGrid.isValidOrPadded(int i, int j, int k) (2). The comment for (1) claims that it excludes padding, whereas (2) is commented as including padding. (1) allows values for 0 <= i < nI, 0 <= j < nJ, 0 <= k < nK and (2) allows values for 0 <= i <= nI, 0 <= j <= nJ, 0 <= k <= nK. So, while there is a difference in how the two deal with nI/nJ/nK values, they deal with 0 values identically.

A third, unused, method SpatialGrid.isValidOrPadded(DiscreteVector dC) allowed values for -1 <= i < nI, -1 <= j < nJ, -1 <= k < nK. This will be removed.

[roughhawkbit]

Also worth noting that DiscreteVector(ContinuousVector cV, Double res) uses Math.ceil of cV.x/res, etc, whereas SpatialGrid.getDiscreteCoordinates(ContinuousVector cC) uses Math.floor.

[roughhawkbit]

SpatialGrid.getValueAt(DiscreteVector dc) returns grid value at (i+1, j+1, k+1) but SpatialGrid.getValueAt(int i, int j, int k) returns grid value at (i, j, k)