Skip to content

MohammedA - #15

Open
mohammedalfakih-dev wants to merge 1 commit into
HackYourAssignment:mainfrom
mohammedalfakih-dev:week11/mohammed-alfakih
Open

MohammedA#15
mohammedalfakih-dev wants to merge 1 commit into
HackYourAssignment:mainfrom
mohammedalfakih-dev:week11/mohammed-alfakih

Conversation

@mohammedalfakih-dev

@mohammedalfakih-dev mohammedalfakih-dev commented Jul 16, 2026

Copy link
Copy Markdown

What I built

How to review

  • Metabase: open the dashboard link above, or see the committed screenshots.
  • Streamlit: see the committed screenshots, or run it yourself with the steps below.
  • Metric definitions: week11-streamlit/metric_definitions.md
  • AI usage: AI_ASSIST.md

How to run the Streamlit app

From a clean clone, with your own Postgres access:

cd week11-streamlit
uv sync
cp .env.example .env      # set your own POSTGRES_URL (with ?sslmode=require) + DB_SCHEMA
uv run streamlit run app.py

Prerequisite: your own fct_trips mart populated in your dev_<name> schema (from Week 10).

Data dependency: these run steps need your own Week 10 mart. A reviewer who does not have that data cannot rebuild your numbers, so the committed screenshots are the canonical evidence of a working submission.

What reviewers should see (expected results)

  • Total trips (fct_trips row count): 56,546
  • Busiest hour of day: 17:00, with 4,576 trips
  • Top payment type by share: Credit card, 36,656 of 56,546 trips (64.8%)
  • Data freshness : latest pickup is 2024-01-31 23:57:29

Known limitations / out of scope

  • None. All Required assignment features are complete and working

Extra completed

  • Metabase date filter on >=2 Questions
  • Streamlit auto-refresh

Self-check

  • bash .hyf/test.sh passes
  • No credentials committed (no password in app.py, .env is gitignored)
  • Screenshots of the Metabase dashboard and the running Streamlit app are committed

@lassebenni lassebenni closed this Jul 17, 2026
@lassebenni lassebenni reopened this Jul 17, 2026
@github-actions

Copy link
Copy Markdown

📝 HackYourFuture auto grade

Assignment Score: 100 / 100 ✅

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

Test Details

=== Week 11 Autograder ===
  ✓ PASS  found README.md
  ✓ PASS  found AI_ASSIST.md
  ✓ PASS  found week11-streamlit/app.py
  ✓ PASS  found week11-streamlit/pyproject.toml
  ✓ PASS  found week11-streamlit/uv.lock
  ✓ PASS  found week11-streamlit/metric_definitions.md
  ✓ PASS  Level 1: required files (20/20 pts)
  ✓ PASS  root .gitignore excludes .env
  ✓ PASS  week11-streamlit/.env not committed
  ✓ PASS  app.py: no hardcoded Postgres credentials found
  ✓ PASS  Level 2: secrets hygiene (15/15 pts)
  ✓ PASS  app.py: imports sqlalchemy
  ✓ PASS  app.py: reads credentials from the environment
  ✓ PASS  app.py: uses .metric() (st.metric or a st.columns() cell)
  ✓ PASS  app.py: queries fct_trips
  ✓ PASS  app.py: uses @st.cache_data
  ✓ PASS  Level 3: Streamlit app content (25/25 pts)
  ✓ PASS  metric_definitions.md: all five fields present
  ✓ PASS  metric_definitions.md: 6 panels documented (>=4 expected: 3 Metabase + 1 Streamlit)
  ✓ PASS  Level 4: metric definitions (20/20 pts)
  ✓ PASS  README.md: dashboard link present under 'My submission'
  ✓ PASS  Level 5: Metabase dashboard documented (10/10 pts)
  ✓ PASS  AI_ASSIST.md: filled (1555 chars, no leftover TODO lines)
  ✓ PASS  Level 6: AI assistance log (10/10 pts)

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

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.

2 participants