From e5739762f232a74f8a4bd66238b68f1ada1c96fc Mon Sep 17 00:00:00 2001 From: Courtney Holcomb Date: Wed, 27 Sep 2023 12:11:23 -0700 Subject: [PATCH] Cleanup --- metricflow/query/query_parser.py | 1 - 1 file changed, 1 deletion(-) diff --git a/metricflow/query/query_parser.py b/metricflow/query/query_parser.py index 1571020e74..17b4139eb1 100644 --- a/metricflow/query/query_parser.py +++ b/metricflow/query/query_parser.py @@ -671,7 +671,6 @@ def _parse_group_by( group_by: Optional[Tuple[GroupByParameter, ...]] = None, ) -> QueryTimeLinkableSpecSet: """Convert the linkable spec names into the respective specification objects.""" - # TODO: refactor to only support group_by object inputs (removing group_by_names param) assert not ( group_by_names and group_by ), "Both group_by_names and group_by were set, but if a group by is specified you should only use one of these!"