[PR #1211] [MERGED] Moved pytest from optional to required dependencies in tinytorch/pyproject.toml #1176

New Issue

GiteaMirror · 2026-03-22T16:03:03-05:00

GiteaMirror commented

2026-03-22 16:03:03 -05:00

📋 Pull Request Information

Original PR: https://github.com/harvard-edge/cs249r_book/pull/1211
Author: @Roldao-Neto
Created: 3/4/2026
Status: ✅ Merged
Merged: 3/4/2026
Merged by: @profvjreddi

Base: main ← Head: pytest-dependency-fix

📝 Commits (1)

7590682 moved pytest from optional to dependencies in TinyTorch/pyproject.toml

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 tinytorch/pyproject.toml (+1 -1)

📄 Description

Fixed pytest dependency issue by editing tinytorch/pyproject.toml

The previous configuration was causing a problem during installation with uv, here is the log of what happened after finishing the installation with curl:

rolds@MacBook-Pro-de-Roldao tinytorch % uv run tito setup
╭──────────────────────────────────────────────╮
│ │
│ Tiny🔥TORCH: Don't import it. Build it. │
│ │
╰──────────────────────────────────────────────╯

╭───────────── Welcome to TinyTorch (Pre-release) ─────────╮
│ 🎓 LEARNING APPROACH │
│ │
│ Solutions are included in the notebooks. This is intentional! │
│ │
│ The best way to learn: │
│ 1. Read the module and run the code │
│ 2. Study how the solutions work │
│ 3. Try implementing from scratch │
│ (reset with: tito module reset) │
│ │
│ 🐛 PRE-RELEASE: We're looking for bugs and feedback! │
│ Found something? → github.com/harvard-edge/cs249r_book/discussions │
╰────────────────────────────────╯

╭──────────────────────────────── Environment Issues ────╮
│ ❌ Environment validation failed: │
│ • Missing dependency: pytest. Run: pip install -r requirements.txt │
│ • Missing dependency: jupytext. Run: pip install -r requirements.txt │
╰───────────────────╯

Run 'tito system health' for detailed diagnosis

╭─────── Welcome to Tiny🔥Torch!──────────────────────╮
│ 🔥 Tiny🔥Torch First-Time Setup │
│ │
│ This will configure your development environment for building ML systems from scratch. │
╰────────────────-------------------------------------────╯
Checking existing setup...

After using uv add pytest:

rolds@MacBook-Pro-de-Roldao tinytorch % uv run tito setup
╭───────────────────────────╮
│ │
│ Tiny🔥TORCH: Don't import it. Build it. │
│ │
╰──────────────────────────────╯
╭───────────────── Welcome to Tiny🔥Torch! ───╮
│ 🔥 Tiny🔥Torch First-Time Setup │
│ │
│ This will configure your development environment for building ML systems from scratch. │
╰─────────----------------------------------------------─╯
Checking existing setup...

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/harvard-edge/cs249r_book/pull/1211 **Author:** [@Roldao-Neto](https://github.com/Roldao-Neto) **Created:** 3/4/2026 **Status:** ✅ Merged **Merged:** 3/4/2026 **Merged by:** [@profvjreddi](https://github.com/profvjreddi) **Base:** `main` ← **Head:** `pytest-dependency-fix` --- ### 📝 Commits (1) - [`7590682`](https://github.com/harvard-edge/cs249r_book/commit/7590682f90f01d4880c470ebd2984e38e423dfc1) moved pytest from optional to dependencies in TinyTorch/pyproject.toml ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `tinytorch/pyproject.toml` (+1 -1) </details> ### 📄 Description # Fixed pytest dependency issue by editing tinytorch/pyproject.toml ## The previous configuration was causing a problem during installation with uv, here is the log of what happened after finishing the installation with curl: rolds@MacBook-Pro-de-Roldao tinytorch % uv run tito setup ╭──────────────────────────────────────────────╮ │ │ │ Tiny🔥TORCH: Don't import it. Build it. │ │ │ ╰──────────────────────────────────────────────╯ ╭───────────── Welcome to TinyTorch (Pre-release) ─────────╮ │ 🎓 LEARNING APPROACH │ │ │ │ Solutions are included in the notebooks. This is intentional! │ │ │ │ The best way to learn: │ │ 1. Read the module and run the code │ │ 2. Study how the solutions work │ │ 3. Try implementing from scratch │ │ (reset with: tito module reset) │ │ │ │ 🐛 PRE-RELEASE: We're looking for bugs and feedback! │ │ Found something? → github.com/harvard-edge/cs249r_book/discussions │ ╰────────────────────────────────╯ ╭──────────────────────────────── Environment Issues ────╮ │ ❌ Environment validation failed: │ │ • Missing dependency: pytest. Run: pip install -r requirements.txt │ │ • Missing dependency: jupytext. Run: pip install -r requirements.txt │ ╰───────────────────╯ Run 'tito system health' for detailed diagnosis ╭─────── Welcome to Tiny🔥Torch!──────────────────────╮ │ 🔥 Tiny🔥Torch First-Time Setup │ │ │ │ This will configure your development environment for building ML systems from scratch. │ ╰────────────────-------------------------------------────╯ Checking existing setup... ## After using uv add pytest: rolds@MacBook-Pro-de-Roldao tinytorch % uv run tito setup ╭───────────────────────────╮ │ │ │ Tiny🔥TORCH: Don't import it. Build it. │ │ │ ╰──────────────────────────────╯ ╭───────────────── Welcome to Tiny🔥Torch! ───╮ │ 🔥 Tiny🔥Torch First-Time Setup │ │ │ │ This will configure your development environment for building ML systems from scratch. │ ╰─────────----------------------------------------------─╯ Checking existing setup... --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2026-03-22 16:03:03 -05:00

GiteaMirror closed this issue

2026-03-22 16:03:03 -05:00

GiteaMirror referenced this issue

2026-04-11 08:04:55 -05:00

[GH-ISSUE #1176] Typo in documentation #1720

GiteaMirror referenced this issue

2026-04-19 12:21:07 -05:00

[GH-ISSUE #1176] Typo in documentation #4340

GiteaMirror referenced this issue

2026-04-21 21:41:39 -05:00

[GH-ISSUE #1176] Typo in documentation #5682