Merge pull request #577 from cmu-delphi/sgratzl/bugfixing

fix: trendseries serialization + require signal in coverage endpoint

Author: sgratzl

src/server/endpoints/covidcast.py

@@ -244,8 +244,8 @@ def handle_trendseries():
     def gen(rows):
         for key, group in groupby((parse_row(row, fields_string, fields_int, fields_float) for row in rows), lambda row: (row["geo_type"], row["geo_value"], row["source"], row["signal"])):
             trends = compute_trends(key[0], key[1], key[2], key[3], shifter, ((row["time_value"], row["value"]) for row in group))
-            for t in trends:
-                yield t
+            for trend in trends:
+                yield trend.asdict()
 
     # execute first query
     try:
@@ -501,7 +501,7 @@ def handle_coverage():
     similar to /signal_dashboard_coverage for a specific signal returns the coverage (number of locations for a given geo_type)
     """
 
-    signal = parse_source_signal_arg("signal")
+    signal = parse_source_signal_pairs()
     geo_type = request.args.get("geo_type", "county")
     if "window" in request.values:
         time_window = parse_day_range_arg("window")
@@ -531,6 +531,7 @@ def handle_coverage():
     q.where_source_signal_pairs("source", "signal", signal)
     q.where_time_pairs("time_type", "time_value", [TimePair("day", [time_window])])
     q.group_by = "c.source, c.signal, c.time_value"
+    q.set_order("source", "signal", "time_value")
 
     _handle_lag_issues_as_of(q, None, None, None)