Opened 2 years ago

Closed 12 months ago

#3302 closed enhancement (fixed)

t.rast.algebra: add support for suffix option in output map names

Reported by: veroandreo Owned by: grass-dev@…
Priority: normal Milestone: 7.6.0
Component: Temporal Version: svn-trunk
Keywords: t.rast.algebra, t.rast.mapcalc, suffix Cc:
CPU: Unspecified Platform: Unspecified

Description

Currently, G7:t.rast.algebra uses "basename_n" (n=1,2,3...number of maps in the strds) as naming scheme for output maps. This makes it a bit difficult to sort them afterwards (#2736), particularly, when one runs some processing not in the temporal framework on them (hence, the output maps are not registered into a strds).

It would be great to also have suffix parameter in t.rast.algebra as well as G7:t.rast.mapcalc, as it is already in G7:t.rast.aggregate with the possibility to have granularity, time or numerical suffix (with zero padding). There are comments on this issue in #2294, but the ticket was for t.rast.aggregate.

Attachments (1)

temporal.diff (24.0 KB) - added by veroandreo 12 months ago.
patch for old grass version to implement suffix

Download all attachments as: .zip

Change History (6)

comment:1 Changed 14 months ago by neteler

Milestone: 7.4.07.4.1

Ticket retargeted after milestone closed

Changed 12 months ago by veroandreo

Attachment: temporal.diff added

patch for old grass version to implement suffix

comment:2 Changed 12 months ago by veroandreo

This was a patch written by Luca some time ago to implement suffix for G7:t.rast.algebra and G7:t.rast.mapcalc, but there were errors in the testsuite.

comment:3 Changed 12 months ago by huhabla

I have implemented the suffix option in t.rast.algebra in grass trunk revision r72478. Please test.

comment:4 Changed 12 months ago by veroandreo

YAY!!! Just tested the 3 different suffix in G7:t.rast.algebra and they work as expected. Thanks! :)

comment:5 Changed 12 months ago by veroandreo

Milestone: 7.4.17.6.0
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.