grass
grass copied to clipboard
r.surf.gauss: Added seed option and flag
Addresses issue https://github.com/OSGeo/grass/issues/1044 to add seed parameter and flag for reproducible results to the r.surf.gauss module. However, tests are currently not present and should still be included.
I know this is a draft, but just to be clear, the compilation fails for raster/r.surf.gauss.
@petrasovaa I updated the tests to check the univar stats are correct,
The Labeler went nearly all-in for this. @echoix Do you perhaps know why?
I think it's because it was an old PR, and merging main in it meant a lot of changes in everywhere. But I'm not sure. Now we don't remove labels automatically, so a next commit should only use the ones that are defined.
If it is still unexpected, maybe a rebase would do the trick...
...merging main in it meant a lot of changes in everywhere....
That sounds plausible.
...merging main in it meant a lot of changes in everywhere....
That sounds plausible.
It doesn't worry that me that much.
@cwhite911 see the discussion we had in https://github.com/OSGeo/grass/pull/3480#discussion_r1519351514, we basically decided not to use the -s flag, so this PR needs to be updated...
...merging main in it meant a lot of changes in everywhere....
That sounds plausible.
It doesn't worry that me that much.
This might of been my fault. When was catching this project up I rebased and had to go back and fix the branch by resetting it to your commit then cherry-picking the work I just did.