Merge pull request #594 from akrherz/windrose_time

tweak windrose text label for clarity
akrherz · Apr 27, 2022 · ec98f40 · ec98f40
2 parents 60c06cf + e10679b
commit ec98f40
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/src/pyiem/windrose_utils.py b/src/pyiem/windrose_utils.py
@@ -65,7 +65,7 @@ def _make_timelimit_string(kwargs):
         d2 = ets.strftime("%b %-d")
         if sts.strftime("%m%d") > ets.strftime("%m%d"):
             d1, d2 = d2, d1
-        tlimit += f"{d1} - {d2}"
+        tlimit += f"{d1} - {d2}, "
     elif months is not None and len(months) < 12:
         for h in months:
             tlimit += f"{month_abbr[h]},"

diff --git a/tests/baseline/test_windrose_month_limiter.png b/tests/baseline/test_windrose_month_limiter.png
diff --git a/tests/test_windrose_utils.py b/tests/test_windrose_utils.py
@@ -96,6 +96,8 @@ def test_windrose_month_limiter():
         drct=drct,
         valid=valid,
         months=[4, 5, 6],
+        limit_by_doy=True,
+        hours=list(range(5, 12)),
         nogenerated=True,
     )
     return fig