[GRASS-dev] [GRASS GIS] #3741: t.rast.series: allow multiple methods/output

GRASS GRASS developer
1

#3741: t.rast.series: allow multiple methods/output
---------------------------+-------------------------
Reporter: sbl | Owner: grass-dev@…
     Type: enhancement | Status: new
Priority: normal | Milestone: 7.8.0
Component: Temporal | Version: svn-trunk
Keywords: t.rast.series | CPU: All
Platform: All |
---------------------------+-------------------------
After r.series received support for multiple methods/outputs in one run,
computing multiple STDS statistics with t.rast.series could get a speed up
if this is handed down to ths module.
Attached a diff where I also tried to incorporate changes in the test
coverage, but I have no experience with the test-suit.
So manual testing of the diff would be welcome. When applied, also manual
should be updated...

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3741&gt;
GRASS GIS <https://grass.osgeo.org>

2

#3741: t.rast.series: allow multiple methods/output
--------------------------+---------------------------
  Reporter: sbl | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: normal | Milestone: 7.8.0
Component: Temporal | Version: svn-trunk
Resolution: | Keywords: t.rast.series
       CPU: All | Platform: All
--------------------------+---------------------------
Changes (by sbl):

* Attachment "t.rast.series_multi.diff" added.

Support for multiple methods / output in t.rast.series

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3741&gt;
GRASS GIS <https://grass.osgeo.org>

3

#3741: t.rast.series: allow multiple methods/output
--------------------------+---------------------------
  Reporter: sbl | Owner: grass-dev@…
      Type: enhancement | Status: closed
  Priority: normal | Milestone: 7.8.0
Component: Temporal | Version: svn-trunk
Resolution: fixed | Keywords: t.rast.series
       CPU: All | Platform: All
--------------------------+---------------------------
Changes (by sbl):

* status: new => closed
* resolution: => fixed

Comment:

In [changeset:"74123" 74123]:
{{{
#!CommitTicketReference repository="" revision="74123"
multi method support for t.rast.series; fix #3741
}}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3741#comment:1&gt;
GRASS GIS <https://grass.osgeo.org>

4

#3741: t.rast.series: allow multiple methods/output
--------------------------+---------------------------
  Reporter: sbl | Owner: grass-dev@…
      Type: enhancement | Status: closed
  Priority: normal | Milestone: 7.8.0
Component: Temporal | Version: svn-trunk
Resolution: fixed | Keywords: t.rast.series
       CPU: All | Platform: All
--------------------------+---------------------------

Comment (by pmav99):

#74123 introduced a {{{SsyntaxError}}} at line 90

{{{
      def test_multi_stats(self):
          self.assertModule("t.rast.series", input="A",
- method=["average", "maximum", "minimum",
+ method=["average", "maximum", "minimum"],

}}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3741#comment:2&gt;
GRASS GIS <https://grass.osgeo.org>

5

#3741: t.rast.series: allow multiple methods/output
--------------------------+---------------------------
  Reporter: sbl | Owner: grass-dev@…
      Type: enhancement | Status: closed
  Priority: normal | Milestone: 7.8.0
Component: Temporal | Version: svn-trunk
Resolution: fixed | Keywords: t.rast.series
       CPU: All | Platform: All
--------------------------+---------------------------

Comment (by neteler):

In [changeset:"74161" 74161]:
{{{
#!CommitTicketReference repository="" revision="74161"
t.rast.series testsuite: missing bracket added (fixes #3741) (contributed
by pmav99)
}}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3741#comment:3&gt;
GRASS GIS <https://grass.osgeo.org>

6

Thanks for both catching and fixing it.
Should not have happened in the first place. I need to improve my routines here, sorry...

Cheers
Stefan

-----Original Message-----
From: GRASS GIS <trac@osgeo.org>
Sent: onsdag 6. mars 2019 09:30
Cc: grass-dev@lists.osgeo.org
Subject: Re: [GRASS GIS] #3741: t.rast.series: allow multiple methods/output

#3741: t.rast.series: allow multiple methods/output
--------------------------+---------------------------
  Reporter: sbl | Owner: grass-dev@…
      Type: enhancement | Status: closed
  Priority: normal | Milestone: 7.8.0
Component: Temporal | Version: svn-trunk
Resolution: fixed | Keywords: t.rast.series
       CPU: All | Platform: All
--------------------------+---------------------------

Comment (by neteler):

In [changeset:"74161" 74161]:
{{{
#!CommitTicketReference repository="" revision="74161"
t.rast.series testsuite: missing bracket added (fixes #3741) (contributed by pmav99) }}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3741#comment:3&gt;
GRASS GIS <https://grass.osgeo.org>