Skip to content

Clarify local taxi_pipeline vs shared namespaced dag_id - #5

Merged
lassebenni merged 1 commit into
mainfrom
clarify-local-vs-shared-dag-id
Jul 23, 2026
Merged

Clarify local taxi_pipeline vs shared namespaced dag_id#5
lassebenni merged 1 commit into
mainfrom
clarify-local-vs-shared-dag-id

Conversation

@lassebenni

Copy link
Copy Markdown
Collaborator

Summary

Makes Task 7 screenshot expectations explicit in the starter README:

  • local Astro keeps taxi_pipeline
  • shared deploy copy must use <yourname>_taxi_pipeline
  • local vs shared screenshots mapped clearly

Curriculum chapter companion: lassebenni/hyf-datatrack branch cursor/week12-clarify-dag-namespace-screenshots-46df.

Make Task 7 screenshot expectations explicit so students keep local
Astro work on taxi_pipeline and only rename the shared deploy copy.
@github-actions

Copy link
Copy Markdown

📝 HackYourFuture auto grade

Assignment Score: 25 / 100 ❌

Status: ❌ Not passed
Minimum score to pass: 60
🧪 The auto grade is experimental and still being improved

Test Details

=== Week 12 Autograder — Orchestrated Pipeline ===
  ✓ PASS  found dags/taxi_pipeline.py
  ✓ PASS  found tests/test_dag_integrity.py
  ✓ PASS  found requirements.txt
  ✓ PASS  found RUNBOOK.md
  ✓ PASS  found ASSIGNMENT_REPORT.md
  ✓ PASS  found AI_ASSIST.md
  ✓ PASS  Level 1: required files (20/20 pts)
  ✗ FAIL  dags/taxi_pipeline.py: raise NotImplementedError still present — the DAG is not implemented
  ✓ PASS  dags/taxi_pipeline.py: defines a @dag with tasks
  ✓ PASS  Level 2: DAG implemented (5/15 pts)
  ✗ FAIL  dags/taxi_pipeline.py: only 1/3 expected tasks found (ingest, dbt_run, dbt_test)
  ✗ FAIL  dags/taxi_pipeline.py: no >> dependency chain found
  ✓ PASS  Level 3: sequential tasks (0/20 pts)
  ✗ FAIL  dags/taxi_pipeline.py: no uvx found — plain 'dbt' crashes on Python 3.14 (see Chapter 4)
  ✗ FAIL  dags/taxi_pipeline.py: no 'retries' found in default_args
  ✓ PASS  Level 4: uvx dbt + retries (0/20 pts)
  ✗ FAIL  dags/taxi_pipeline.py: no {{ ds }} / logical_date / get_current_context found — partition must come from the run date
  ✗ FAIL  dags/taxi_pipeline.py: catchup=False not found — required for safe normal operation
  ✓ PASS  Level 5: parameterized runs (0/15 pts)
  ✗ FAIL  RUNBOOK.md: still a template (374 chars, 7 TODO marker(s)) — fill in all four sections
  ✗ FAIL  AI_ASSIST.md: still a template (301 chars, 4 TODO marker(s))
  ✓ PASS  Level 6: documentation (0/10 pts)

Score: 25 / 100  (passing: 60)  pass=false

Reminder: the shared-Airflow deploy, the green run, and backfill
idempotency are Target-tier items a teacher reviews by hand — a high
static score here is necessary but not sufficient for Target.

@lassebenni
lassebenni merged commit 5f9f234 into main Jul 23, 2026
0 of 2 checks passed
@lassebenni
lassebenni deleted the clarify-local-vs-shared-dag-id branch July 23, 2026 13:33
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