From c1e10c4627f91871dd4c72f8a662c200d5d4b3e9 Mon Sep 17 00:00:00 2001 From: Ville Brofeldt <33317356+villebro@users.noreply.github.com> Date: Wed, 10 Feb 2021 21:45:45 +0200 Subject: [PATCH] fix: sorting by saved metric (#13059) --- superset/connectors/sqla/models.py | 2 ++ superset/examples/birth_names.py | 7 ++++--- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/superset/connectors/sqla/models.py b/superset/connectors/sqla/models.py index b7204fed9e..a64a62869f 100644 --- a/superset/connectors/sqla/models.py +++ b/superset/connectors/sqla/models.py @@ -1150,6 +1150,8 @@ class SqlaTable( # pylint: disable=too-many-public-methods,too-many-instance-at col = self.adhoc_metric_to_sqla(col, columns_by_name) elif col in columns_by_name: col = columns_by_name[col].get_sqla_col() + elif col in metrics_by_name: + col = metrics_by_name[col].get_sqla_col() if isinstance(col, Label): label = col._label # pylint: disable=protected-access diff --git a/superset/examples/birth_names.py b/superset/examples/birth_names.py index a8729940dc..583a0991f3 100644 --- a/superset/examples/birth_names.py +++ b/superset/examples/birth_names.py @@ -271,8 +271,8 @@ def create_slices( groupby=["name"], adhoc_filters=[gen_filter("gender", "girl")], row_limit=50, - timeseries_limit_metric="sum__num", - metrics=metrics, + timeseries_limit_metric=metric, + metrics=[metric], ), ), Slice( @@ -300,7 +300,8 @@ def create_slices( groupby=["name"], adhoc_filters=[gen_filter("gender", "boy")], row_limit=50, - metrics=metrics, + timeseries_limit_metric=metric, + metrics=[metric], ), ), Slice(