better human times fixes #420

Riverscapes · Sep 8, 2021 · 9bfe728 · 9bfe728
1 parent f97b50b
commit 9bfe728
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/lib/commons/rscommons/util.py b/lib/commons/rscommons/util.py
@@ -235,7 +235,7 @@ def pretty_duration(time_s=False):
     """
     if not time_s >= 0:
         return '???'
-    seconds = time_s % 60
+    seconds = floor(time_s % 60)
     minutes = floor(time_s / 60) % 60
     hours = floor(time_s / 3600) % 24
     if time_s < 60:

diff --git a/lib/commons/test/test_util.py b/lib/commons/test/test_util.py
@@ -25,6 +25,9 @@ def test_pretty_duration(self):
         test1 = util.pretty_duration(100)
         self.assertEqual(test1, '1:40 minutes')
 
+        test1 = util.pretty_duration(103.234234)
+        self.assertEqual(test1, '1:43 minutes')
+
         test1 = util.pretty_duration(1000)
         self.assertEqual(test1, '16:40 minutes')