Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#3741 closed enhancement (fixed)

t.rast.series: allow multiple methods/output

Reported by: sbl Owned by: grass-dev@…
Priority: normal Milestone: 7.8.0
Component: Temporal Version: svn-trunk
Keywords: t.rast.series Cc:
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...

Attachments (1)

t.rast.series_multi.diff (6.2 KB) - added by sbl 2 years ago.
Support for multiple methods / output in t.rast.series

Download all attachments as: .zip

Change History (4)

Changed 2 years ago by sbl

Attachment: t.rast.series_multi.diff added

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

comment:1 Changed 2 years ago by sbl

Resolution: fixed
Status: newclosed

In 74123:

multi method support for t.rast.series; fix #3741

comment:2 Changed 2 years ago by pmav99

It seems that the previous commit introduced a SyntaxError at line 90. Please fix:

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

Last edited 2 years ago by pmav99 (previous) (diff)

comment:3 Changed 2 years ago by neteler

In 74161:

t.rast.series testsuite: missing bracket added (fixes #3741) (contributed by pmav99)

Note: See TracTickets for help on using tickets.