Minor updates in raster resampling section #660
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Some minor suggestions, please have a look to judge what is actually better π. It's a well understandable and didactic section @Nowosad π!
Especially check the small additional explanations in order to distinguish cubic interpolation and cubic spline interpolation - since you already added 'splines (polynomial functions)' to explain the latter. Main source is Wikipedia (and see also dsp stackexchange) since it's not explained in
terra::resample()
. I didn't dig further behind the scenes interra::resample()
, so best double check. BTW according to Wikipedia, there's also bicubic interpolation, used for 2D interpolation, which could be the algorithm used here (unsure).Beside that, maybe check how the terms 'grid' and 'raster' are used in the book. I assume each has its own meaning, but better don't unnecessarily mix them, since it then seems intended and may be confusing. Cf. below extract from chapter 2, where (I think) the 'grid' term is introduced and is regarded a more general concept than 'raster'. In this PR, 'grid' is replaced by 'raster'.