Skip to content

Commit

Permalink
[daggy-u] [dbt] - Correct model in Lesson 6 (DEV-180) (#20371)
Browse files Browse the repository at this point in the history
## Summary & Motivation

This PR removes a problematic column in the `daily_metrics` model in
Lesson 6 of the Dagster & dbt course. It was causing issues with the
`group by` clause.

## How I Tested These Changes

dagster dev, build/run dbt, local materialization
  • Loading branch information
erinkcochran87 authored Mar 8, 2024
1 parent e3d64fd commit e6537ff
Showing 1 changed file with 0 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ with
select
date_trunc('day', pickup_datetime) as date_of_business,
count(*) as trip_count,
count(*) - lag(count(*), 1) over (order by date_trunc('day', pickup_datetime)) as trip_count_change,
sum(duration) as total_duration,
sum(duration) / count(*) as average_duration,
sum(total_amount) as total_amount,
Expand Down

1 comment on commit e6537ff

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deploy preview for dagster-docs ready!

✅ Preview
https://dagster-docs-bhdusm96w-elementl.vercel.app
https://master.dagster.dagster-docs.io

Built with commit e6537ff.
This pull request is being automatically deployed with vercel-action

Please sign in to comment.