#2294: t.rast.aggregate: Name output rasters based on start-time
------------------------------+---------------------------------------------
Reporter: sbl | Owner: grass-dev@…
Type: enhancement | Status: new
Priority: normal | Milestone: 7.1.0
Component: Default | Version: svn-trunk
Keywords: t.rast.aggregate | Platform: Unspecified
Cpu: Unspecified |
------------------------------+---------------------------------------------
Using a numerical suffix for aggregated time series data makes it less
intuitive to use the results outside the TGRASS framework. One has to look
up the time stamp for e.g. a rastermap "temperature_avg_months_317" in
order to know which time periode it refers to (even those people who
produced it), while "temperature_avg_months_1989_09" already tells
something.
Please find attached a proposal for naming output rasters based on start
time - truncated according to granularity, implemented in a flag (s).
The attached patch is a modification of a patch I made for GRAS 7.0 wich I
currently use and where it works fine for absolute time datasets. Not sure
if it is suitable for relative or cyclic time though and it has not been
tested with GRASS 7.1. So it needs a critical review in any case.
I do not know if such a solution would be relevant for t.rast.accumulate
too (I have not used the module yet)...
If you find it useful I can send a patch for GRASS 7.0 too.
#2294: t.rast.aggregate: Name output rasters based on start-time
------------------------------+---------------------------------------------
Reporter: sbl | Owner: grass-dev@…
Type: enhancement | Status: new
Priority: normal | Milestone: 7.1.0
Component: Temporal | Version: svn-trunk
Keywords: t.rast.aggregate | Platform: All
Cpu: All |
------------------------------+---------------------------------------------
Comment(by huhabla):
Many thanks for your patches.
I have integrated it in r61066 and modified the aggregation methods to
allow parallel processing and faster metadata management. The new
aggregation methods are experimental, hence no backport to 7.0 is planned
right now. Further testing is needed.
Your map name suffix approach is now available as library method and can
be reused in other modules as well.
I am reopening this ticket because there are unsolved issues here.
* Can this patch handle relative timestamp?
* We should keep option to choose between the temporal and numerical
suffix. Possibly as a parameter?
* With numerical suffix, it should pad the number with zeros.
* This should be applied to all t.* modules generating new maps.
#2294: t.rast.aggregate: Name output rasters based on start-time
--------------------------+-------------------------------------------------
Reporter: sbl | Owner: grass-dev@…
Type: enhancement | Status: reopened
Priority: normal | Milestone: 7.1.0
Component: Temporal | Version: svn-trunk
Resolution: | Keywords: t.rast.aggregate
Platform: All | Cpu: All
--------------------------+-------------------------------------------------
Comment(by annakrat):
Replying to [comment:4 annakrat]:
> I am reopening this ticket because there are unsolved issues here.
>
> * Can this patch handle relative timestamp?
> * We should keep option to choose between the temporal and numerical
suffix. Possibly as a parameter?
> * With numerical suffix, it should pad the number with zeros.
I did that in r61888.
> * This should be applied to all t.* modules generating new maps.
>