t.rast.algebra temporal neighbourhood modifier not clearly documented

[mankoff]

trafficstars

The temporal neighbourhood modifier states The format is strds[t,r,c], where t is the temporal offset, r is the row offset and c is the column offset but never explicitly states what behavior occurs if <3 elements are provided.

From the example, it appears 1 value is assumed to be t, 2 is r,c, and 3 is t,r,c? This is confusing, as the 1st element is sometimes treated as t and other times as r. It would be good to explicitly state this if this is the case.

https://github.com/OSGeo/grass/blob/7d9a0da4fcf603bf690e470176a488e2def05d49/temporal/t.rast.algebra/t.rast.algebra.html#L444-L475

[neteler]

Shall this still go into the upcoming 7.8.5?

[ninsbl]

#905 adds new temporal functions and updates documentation. Should I take the documentation part related to this issue out of the PR so it can go into 7.8.6 separately?

[veroandreo]

Will the new feature be available only in grass 8.0? If yes, then the clarifications in documentation related to this issue should probably go in a separate PR, IMO

[neteler]

#905 adds new temporal functions and updates documentation. Should I take the documentation part related to this issue out of the PR so it can go into 7.8.6 separately?

If you could separate out the doc updates (relevant for G7.8.6) in a new PR... thanks

[neteler]

Bumped to 8.0.1

[ninsbl]

Should be backported to 8.2 (and also 7.8)

[ninsbl]

This got fixed with #905 in 8.3. Given that 8.4 is on it`s way I close this issue (as backporting to GRASS GIS 7 is not planned).