[GRASS-dev] [GRASS GIS] #2625: r.stats's nsteps is not region-sensitive

[Paulo van Breugel]

On 14-09-15 11:39, GRASS GIS wrote:
> #2625: r.stats's nsteps is not region-sensitive
> --------------------------+-----------------------------
>    Reporter:  martinl      |      Owner:  grass-dev@…
>        Type:  defect       |     Status:  new
>    Priority:  major        |  Milestone:  7.0.2
>   Component:  Raster       |    Version:  svn-trunk
> Resolution:               |   Keywords:  r.stats, nsteps
>         CPU:  Unspecified  |   Platform:  Unspecified
> --------------------------+-----------------------------
>
> Comment (by mlennert):
>
>   Replying to [comment:2 martinl]:
>   > Any opinion / feedback? Thanks.
>
>   Both region-specific bins and map-specific bins can be useful. The latter
>   allow easier comparison between different subregions, whereas the former
>   allow getting the number of bins asked for in each subregion.
>
>   So, the ideal solution IMHO would be to have both.
+1 Between the two options, I would prefer it to respect the region 
(following the common approach in GRASS). But I agree that having both 
options would even be better.
> In the meantime, the
>   fact that bins are calculated on the entire map's value range should be at
>   least documented explicitly.

The documentation now says that "analysis uses the current geographic 
region (g.region) and mask settings (r.mask)". So the fact that /nsteps/ 
does not respects this is an exception apparently (and it is an 
exception to the general GRASS GIS approach), which should indeed by 
very clearly documented.
>
> --
> Ticket URL: <https://trac.osgeo.org/grass/ticket/2625#comment:3>
> GRASS GIS <https://grass.osgeo.org>
>
> _______________________________________________
> grass-dev mailing list
> grass-dev at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/grass-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/grass-dev/attachments/20150914/932b03b1/attachment.html>