Opened 5 years ago

Closed 5 years ago

Last modified 5 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 5 years ago.
Support for multiple methods / output in t.rast.series

Download all attachments as: .zip

Change History (4)

by sbl, 5 years ago

Attachment: t.rast.series_multi.diff added

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

comment:1 by sbl, 5 years ago

Resolution: fixed
Status: newclosed

In 74123:

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

comment:2 by pmav99, 5 years ago

Unfortunately 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"],

Version 2, edited 5 years ago by pmav99 (previous) (next) (diff)

comment:3 by neteler, 5 years ago

In 74161:

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

Note: See TracTickets for help on using tickets.