Skip to content

Muna#4

Open
MunaNasher wants to merge 1 commit into
HackYourAssignment:mainfrom
MunaNasher:main
Open

Muna#4
MunaNasher wants to merge 1 commit into
HackYourAssignment:mainfrom
MunaNasher:main

Conversation

@MunaNasher

Copy link
Copy Markdown

No description provided.

@lassebenni lassebenni closed this Jun 10, 2026
@lassebenni lassebenni reopened this Jun 10, 2026
@github-actions

Copy link
Copy Markdown

📝 HackYourFuture auto grade

Assignment Score: 50 / 100 ❌

Status: ❌ Not 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
  ✗ FAIL  pipeline.py does not use 'from contextlib import closing' + 'with closing(...)' (Chapter 4 pattern)
  ✓ PASS  pipeline.py silences the azure SDK logger
  ✓ PASS  Level 4: pipeline shape (10/15 pts)
  ✗ FAIL  pipeline.py does not use ON CONFLICT ... DO UPDATE (idempotent upsert)
  ✓ PASS  pipeline.py uses %s placeholders in execute() (parameterised SQL)
  ✓ PASS  Level 5: idempotent upsert (5/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 is missing one of the three required sections
  ✓ PASS  Level 7: AI report (0/10 pts)
  ✗ FAIL  README.md is missing the '## Verification' heading (Task 5)
  ✓ PASS  Level 8: README verification (0/10 pts)
  ✗ FAIL  docs/execution_history.png not found (Task 5 deliverable)
  ✓ PASS  Level 9: execution screenshot (0/10 pts)
  ✓ PASS  .gitignore correctly excludes __pycache__/, *.pyc, and .env

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

@danlaudk

Copy link
Copy Markdown

i'm interested, also interesting for you to remember: which were the failures in the job that only failed when run as a container, but not failing locally?

@danlaudk

danlaudk commented Jun 16, 2026

Copy link
Copy Markdown

the homework was to build on top of your week 5

This week you take the containerized pipeline from Week 5 and deploy it

working on the wrong data, and pushing the wrong dataset is not good. make sure to read the assignment

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.

3 participants