Opened 7 years ago

Closed 6 years ago

#3301 closed enhancement (fixed)

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

Reported by: veroandreo Owned by: grass-dev@…
Priority: normal Milestone: 7.4.0
Component: Addons Version: 7.2.0
Keywords: r.seasons Cc:
CPU: Unspecified Platform: Unspecified

Description

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.

Change History (6)

comment:1 by martinl, 7 years ago

Milestone: 7.2.17.2.2

in reply to:  description ; comment:2 by mmetz, 7 years ago

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.

in reply to:  2 ; comment:3 by neteler, 7 years ago

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.

in reply to:  3 comment:4 by mmetz, 7 years ago

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.

comment:5 by martinl, 7 years ago

Milestone: 7.2.27.4.0

All enhancement tickets should be assigned to 7.4 milestone.

comment:6 by veroandreo, 6 years ago

Resolution: fixed
Status: newclosed

Thanks much! Closing

Note: See TracTickets for help on using tickets.