Hannah

[hannahwn]

No description provided.

[github-actions]

📝 HackYourFuture auto grade

Assignment Score: 80 / 100 ✅

Status: ✅ Passed
Minimum score to pass: 60
🧪 The auto grade is experimental and still being improved

Test Details

=== Week 6 Autograder ===
  ✓ PASS  found Dockerfile
  ✓ PASS  found requirements.txt
  ✓ PASS  found src/pipeline.py
  ✓ PASS  found AI_ASSIST.md
  ✓ PASS  found README.md
  ✓ PASS  found docs/ directory
  ✓ PASS  Level 1: required files (10/10 pts)
  ✓ PASS  requirements.txt pins azure-storage-blob
  ✓ PASS  requirements.txt pins psycopg2-binary
  ✓ PASS  Level 2: pinned dependencies (10/10 pts)
  ✓ PASS  Dockerfile uses a python:3.11 base image
  ✓ PASS  Dockerfile copies requirements before src/ (layer cache stays warm)
  ✓ PASS  Dockerfile has a real CMD instruction
  ✓ PASS  Level 3: Dockerfile (10/10 pts)
  ✓ PASS  pipeline.py reads POSTGRES_URL and AZURE_STORAGE_CONNECTION_STRING from env
  ✓ PASS  pipeline.py uses contextlib.closing() to wrap the Postgres connection
  ✓ PASS  pipeline.py silences the azure SDK logger
  ⚠ WARN  pipeline.py still contains 'raise NotImplementedError' — finish the stubs before submitting
  ✓ PASS  Level 4: pipeline shape (15/15 pts)
  ✓ PASS  pipeline.py uses an upsert (ON CONFLICT ... DO UPDATE)
  ✗ FAIL  pipeline.py does not use %s placeholders for parameterised SQL
  ✓ PASS  Level 5: idempotent upsert (10/15 pts)
  ✗ FAIL  no mention of sslmode=require in the repo — Azure Postgres rejects connections without it
  ✓ PASS  pipeline.py uses the azure-storage-blob SDK (BlobServiceClient)
  ✓ PASS  Level 6: connection + SDK (5/10 pts)
  ✗ FAIL  AI_ASSIST.md has section headers but is not filled in (1963 chars, 3 TODO line(s); target 1800+ chars, 0 TODOs)
  ✓ PASS  Level 7: AI report (0/10 pts)
  ✓ PASS  README.md has '## Verification' heading and embeds a docs/ image
  ✓ PASS  Level 8: README verification (10/10 pts)
  ✓ PASS  execution-history screenshot present at execution_history.png (341941 bytes)
  ✓ PASS  Level 9: execution screenshot (10/10 pts)
  ✓ PASS  .gitignore correctly excludes __pycache__/, *.pyc, and .env

Score: 80 / 100  (passing: 60)  pass=true

[qiraahmad]

Can be removed, as DROP TABLE IF EXISTS wipes all data on every run, which defeats idempotent reruns. Keep only CREATE TABLE IF NOT EXISTS and rely on your upsert below.

[qiraahmad]

remove TODOs

[qiraahmad]

you can reference the screenshot here

[qiraahmad]

Well done on implementing the pipeline implemented! One point I would add is that in week 6 asked you to extend your Week 5 pipeline, but this submission uses the Week 6 scaffold weather data instead, with no mention of that in the PR or AI_ASSIST. 3/5