Skip to content

Issue #308 - Fix Dockerfile apt install failures for Phase 0 Lesson 7#310

Open
tudoranghelina91 wants to merge 1 commit into
rohitg00:mainfrom
tudoranghelina91:308-fix-dockerfile
Open

Issue #308 - Fix Dockerfile apt install failures for Phase 0 Lesson 7#310
tudoranghelina91 wants to merge 1 commit into
rohitg00:mainfrom
tudoranghelina91:308-fix-dockerfile

Conversation

@tudoranghelina91

@tudoranghelina91 tudoranghelina91 commented Jun 24, 2026

Copy link
Copy Markdown

What this PR does

This PR adresses the apt package install failures when building the Dockerfile for Phase 0 / Lesson 7

Kind of change

  • New lesson
  • Fix to an existing lesson
  • Translation
  • New output (prompt, skill, agent, MCP server)
  • Docs / website / tooling

Checklist

  • Code runs without errors with the listed dependencies
  • No comments in code files (docs explain, code is self-explanatory)
  • Built from scratch first, then shown with a framework (for new lessons)
  • Lesson folder matches LESSON_TEMPLATE.md structure
  • ROADMAP.md row for the lesson is a markdown link ([Name](phases/...)), not bare text
  • One lesson per commit (atomic per-lesson rule)
  • Tested locally / code output matches what docs/en.md claims

Phase / lesson

Notes for reviewer

The package repository used doesn't have python3.12 in it's repositories

This PR adresses the apt package install failures when building the Dockerfile for Phase 0 / Lesson 7
@coderabbitai

coderabbitai Bot commented Jun 24, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The Dockerfile and its matching documentation snippet switch the installed Python toolchain from 3.12 to 3.11 (apt packages, venv, dev, and update-alternatives target). The PyTorch install is updated to torch==2.4.1 (from 2.3.1), with torchvision and torchaudio no longer pinned to explicit versions.

Changes

Python 3.11 and PyTorch 2.4.1 Dockerfile Update

Layer / File(s) Summary
Python version and PyTorch pins
phases/00-setup-and-tooling/07-docker-for-ai/code/Dockerfile, phases/00-setup-and-tooling/07-docker-for-ai/docs/en.md
apt packages changed from python3.12/python3.12-venv/python3.12-dev to python3.11 equivalents; update-alternatives target updated to /usr/bin/python3.11; torch bumped from 2.3.1 to 2.4.1; explicit version pins removed from torchvision and torchaudio. Documentation snippet in en.md updated to match.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly matches the main change: fixing Phase 0 Lesson 7 Dockerfile apt install failures.
Description check ✅ Passed The description matches the changeset, which updates the Phase 0 / Lesson 7 Dockerfile to fix package installation failures.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@tudoranghelina91 tudoranghelina91 changed the title Issue #308 Issue #308 - Fix Dockerfile apt install failures for Phase 0 Lesson 7 Jun 24, 2026
@tudoranghelina91

Copy link
Copy Markdown
Author

@rohitg00 I was wondering if this PR could be reviewed please?

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