docs+chore: submit to 'Week 11 Submissions' collection + pin Python 3.12 - #4
Merged
Merged
Conversation
Non-admin students can't create public links and personal collections are admin-only, so teachers can't grade a dashboard left in a personal collection. Instruct students to save into the shared 'Week 11 Submissions' collection. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Without a .python-version, uv sync grabs the student's default interpreter; on Python 3.13/3.14 psycopg2-binary 2.9.9 has no wheels and the build fails. Pin 3.12 to match nyc-taxi-streamlit-reference. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
lassebenni
changed the title
"My submission" now asks for the presentation recording link with a privacy warning (never public, never commit the .mp4), and points to the optional Azure-native path (upload to the shared student-submissions container, share a read-only link) documented in the Week 11 Assignment chapter. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The student-submissions container is shared, so the README now states the naming convention (week-11/<your-name>.mp4) alongside the Azure pointer. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Ship .github/PULL_REQUEST_TEMPLATE.md so the PR body auto-fills (what built / how to review / how to run / extra / self-check), and add a README section on reviewer-ready, reproducible submissions: run from a clean clone against the reviewer's own Postgres, and commit screenshots for the private dev_<name> data and Metabase Questions a reviewer cannot run. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Drop the Loom / YouTube / Drive hosting alternatives; students record locally and upload the .mp4 to the shared student-submissions blob container. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Direct, tenant-scoped Storage browser link to the student-submissions container for students who upload by hand instead of via the az CLI. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two Week 11 assignment-repo fixes:
Week 11 Submissionscollection (Teachers group has view access) instead of a personal collection, which teachers can't see.week11-streamlit/.python-version=3.12. Without ituv syncgrabs the student's default interpreter; on 3.13/3.14psycopg2-binaryhas no wheels and the build fails. Matches nyc-taxi-streamlit-reference.🤖 Generated with Claude Code