[GRASS-dev] [GRASS GIS] #3301: r.seasons: add option to use an input map with per-cell thresholds

#3301: r.seasons: add option to use an input map with per-cell thresholds
-------------------------+-------------------------
Reporter: veroandreo | Owner: grass-dev@…
     Type: enhancement | Status: new
Priority: normal | Milestone: 7.2.1
Component: Addons | Version: 7.2.0
Keywords: r.seasons | CPU: Unspecified
Platform: Unspecified |
-------------------------+-------------------------
The possibility to use an input map with per-cell thresholds in r.seasons
would be very useful (not necesarily to replace the current threshold
parameter, but as a different parameter). This would allow to pass a map
of thresholds representing the median of the input time series in each
pixel, for example.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3301&gt;
GRASS GIS <https://grass.osgeo.org>

#3301: r.seasons: add option to use an input map with per-cell thresholds
--------------------------+-------------------------
  Reporter: veroandreo | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: normal | Milestone: 7.2.2
Component: Addons | Version: 7.2.0
Resolution: | Keywords: r.seasons
       CPU: Unspecified | Platform: Unspecified
--------------------------+-------------------------

Comment (by mmetz):

Replying to [ticket:3301 veroandreo]:
> The possibility to use an input map with per-cell thresholds in
r.seasons would be very useful (not necesarily to replace the current
threshold parameter, but as a different parameter). This would allow to
pass a map of thresholds representing the median of the input time series
in each pixel, for example.

A new option `tmap` to use a raster map with per-cell threshold values has
been added in r71070. The `threshold` option has been renamed to `tval`.
Either `tval` or `tmap` must be provided.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3301#comment:2&gt;
GRASS GIS <https://grass.osgeo.org>

#3301: r.seasons: add option to use an input map with per-cell thresholds
--------------------------+-------------------------
  Reporter: veroandreo | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: normal | Milestone: 7.2.2
Component: Addons | Version: 7.2.0
Resolution: | Keywords: r.seasons
       CPU: Unspecified | Platform: Unspecified
--------------------------+-------------------------

Comment (by neteler):

Replying to [comment:2 mmetz]:
> The `threshold` option has been renamed to `tval`. Either `tval` or
`tmap` must be provided.

Shy suggestion: to follow the G7 parameter name scheme, something with
_value would be better. Like in r.sun or other modules:

{{{
             aspect Name of the input aspect map (terrain aspect or
azimuth of the solar panel) [decimal degrees]
       aspect_value A single value of the orientation (aspect), 270 is
south
                      default: 270
              slope Name of the input slope raster map (terrain slope or
solar panel inclination) [decimal degrees]
        slope_value A single value of inclination (slope)
                      default: 0.0
}}}

AFAIK "tval" whould still work due to the parser magic with underscores.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3301#comment:3&gt;
GRASS GIS <https://grass.osgeo.org>

#3301: r.seasons: add option to use an input map with per-cell thresholds
--------------------------+-------------------------
  Reporter: veroandreo | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: normal | Milestone: 7.2.2
Component: Addons | Version: 7.2.0
Resolution: | Keywords: r.seasons
       CPU: Unspecified | Platform: Unspecified
--------------------------+-------------------------

Comment (by mmetz):

Replying to [comment:3 neteler]:
> Replying to [comment:2 mmetz]:
> > The `threshold` option has been renamed to `tval`. Either `tval` or
`tmap` must be provided.
>
> Shy suggestion: to follow the G7 parameter name scheme, something with
_value would be better.

OK, `tval` has been changed to `threshold_value` and `tmap` has been
changed to `threshold_map` in r71078.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3301#comment:4&gt;
GRASS GIS <https://grass.osgeo.org>

#3301: r.seasons: add option to use an input map with per-cell thresholds
--------------------------+-------------------------
  Reporter: veroandreo | Owner: grass-dev@…
      Type: enhancement | Status: closed
  Priority: normal | Milestone: 7.4.0
Component: Addons | Version: 7.2.0
Resolution: fixed | Keywords: r.seasons
       CPU: Unspecified | Platform: Unspecified
--------------------------+-------------------------
Changes (by veroandreo):

* status: new => closed
* resolution: => fixed

Comment:

Thanks much! Closing

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3301#comment:6&gt;
GRASS GIS <https://grass.osgeo.org>