t.rast.algebra temporal neighbourhood modifier not clearly documented

[mankoff]

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.

grass/temporal/t.rast.algebra/t.rast.algebra.html

Lines 444 to 475 in 7d9a0da

	<h3>Temporal neighbourhood modifier</h3>
	The neighbourhood modifier of <em>r.mapcalc</em> is extended for the temporal
	raster algebra with the temporal dimension. The format is strds[t,r,c],
	where t is the temporal offset, r is the row offset and c is the column
	offset.
	<pre class="code">
	strds[2]
	</pre>
	refers to the second successor of the current map.
	<p>
	<pre class="code">
	strds[1,2]
	</pre>
	refers to the cell one row below and two columns to the right of the current
	cell in the current map.
	<p>
	<pre class="code">
	strds[1,-2,-1]
	</pre>
	refers to the cell two rows above and one column to the left of the current
	cell of the first successor map.
	<p>
	<pre class="code">
	strds[-2,0,1]
	</pre>
	refers to the cell one column to the right of the current cell in the
	second predecessor map.

[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