data-engineering-zoomcamp icon indicating copy to clipboard operation
data-engineering-zoomcamp copied to clipboard

dlt workshop q4 issue

Open aturevich opened this issue 4 months ago • 2 comments

Hello there, there is high chance that there is mistake in q4 for dlt workshop

Here are my points. In the end we have to end with tables, excluding blue lines, as they come from first generator and should be updated with values from second generator image

Tried couples of approaches in code, but ended with same answer. If we try to add up age values manually, than no answer will match our result.

image

Probably form needs update

aturevich avatar Feb 07 '24 19:02 aturevich

@adrianbr can you please take a look?

alexeygrigorev avatar Feb 07 '24 20:02 alexeygrigorev

The problem was described in https://github.com/DataTalksClub/data-engineering-zoomcamp/pull/461

The code used to procure the solutions erroneously used different table names for merging, which caused the resulting sum to be 213 (age values from second generator only), which apparently is what is used by the form now.

askeladden31 avatar Feb 08 '24 10:02 askeladden31

Answers were fixed. Closing.

aturevich avatar Feb 09 '24 10:02 aturevich