r.seasons: add option to use an input map with per-cell thresholds

[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.

[mmetz]

Replying to 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.

[neteler]

Replying to 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.

[mmetz]

Replying to neteler:

Replying to 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.

[martinl]

All enhancement tickets should be assigned to 7.4 milestone.

[veroandreo]

Thanks much! Closing