Skip to content

Commit

Permalink
Fix import of TimeGranularity enum in filter_renderer.py
Browse files Browse the repository at this point in the history
This import worked because there is a time_granularity file in MetricFlow
and it imports TimeGranularity from dbt_semantic_interfaces (thus making
it available). However importing it from MetricFlow in dbt_semantic_interfaces
was problematic for multiple reasons
1) It created a circular dependency, and the ProcessPoolExecutor or model
validator freaked out about this
2) dbt_semantic_interfaces cannot depend on MetricFlow
  • Loading branch information
QMalcolm committed May 11, 2023
1 parent d603ada commit 0a070b8
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
import jinja2

from dbt_semantic_interfaces.references import DimensionReference, EntityReference
from metricflow.time.time_granularity import TimeGranularity
from dbt_semantic_interfaces.objects.time_granularity import TimeGranularity


@dataclass(frozen=True)
Expand Down

0 comments on commit 0a070b8

Please sign in to comment.