Skip to content

Fix Dagre layout: split dbt snapshots into own group#188

Merged
vbalalian merged 1 commit intomainfrom
fix/dagster-omni-layout-rank-overlap
Mar 12, 2026
Merged

Fix Dagre layout: split dbt snapshots into own group#188
vbalalian merged 1 commit intomainfrom
fix/dagster-omni-layout-rank-overlap

Conversation

@vbalalian
Copy link
Copy Markdown
Owner

Summary

  • The omni_analytics group rendered side-by-side with dbt_models because snap_user_rfm (at Dagre rank 6 in dbt_models) shared the same rank as all omni_analytics assets, causing Dagre to lay the groups out horizontally instead of vertically.
  • Uses conditional Jinja in the dbt translation config to assign snapshots to a dbt_snapshots group, keeping dbt_models at ranks 1–5 so omni_analytics flows below it.

Test plan

  • CI passes (dg list defs, dbt build)
  • Verify in Dagster UI that omni_analytics group renders below dbt_models instead of beside it
  • Confirm snap_user_rfm appears in a dbt_snapshots group

The omni_analytics group rendered side-by-side with dbt_models because
snap_user_rfm (rank 6 in dbt_models) shared the same Dagre rank as
omni_analytics assets. Moving snapshots to a dbt_snapshots group keeps
dbt_models at ranks 1-5 so omni_analytics flows cleanly below.
@vbalalian vbalalian merged commit 8a2f05e into main Mar 12, 2026
3 checks passed
@vbalalian vbalalian deleted the fix/dagster-omni-layout-rank-overlap branch March 12, 2026 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant