[PR #1262] [MERGED] Fix #1221 suppress expected overflow warning in test #3789

New Issue

GiteaMirror · 2026-04-16T01:58:14-05:00

GiteaMirror commented

2026-04-16 01:58:14 -05:00

📋 Pull Request Information

Original PR: https://github.com/harvard-edge/cs249r_book/pull/1262
Author: @asgalon
Created: 3/20/2026
Status: ✅ Merged
Merged: 3/20/2026
Merged by: @profvjreddi

Base: dev ← Head: feature/feature-1221

📝 Commits (1)

bf5e3ce Fix #1221 suppress expected overflow warning in test

📊 Changes

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

View changed files

📝 tinytorch/src/02_activations/02_activations.py (+6 -1)

📄 Description

Fixes issue #1221 . Replaced slightly confusing warning message about expected overflow in exp on tensor with large numerical values) with message saying that the warning was suppressed. So it does no longer show a scary warning in red, but still acknowledges the overflow happening.

By submitting this PR, you agree to release your contribution under the project's CC-BY-NC-ND license.

_{🔄 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/1262 **Author:** [@asgalon](https://github.com/asgalon) **Created:** 3/20/2026 **Status:** ✅ Merged **Merged:** 3/20/2026 **Merged by:** [@profvjreddi](https://github.com/profvjreddi) **Base:** `dev` ← **Head:** `feature/feature-1221` --- ### 📝 Commits (1) - [`bf5e3ce`](https://github.com/harvard-edge/cs249r_book/commit/bf5e3cecec6bdd27cd9d571799cce68e33a02084) Fix #1221 suppress expected overflow warning in test ### 📊 Changes **1 file changed** (+6 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `tinytorch/src/02_activations/02_activations.py` (+6 -1) </details> ### 📄 Description Fixes issue #1221 . Replaced slightly confusing warning message about expected overflow in exp on tensor with large numerical values) with message saying that the warning was suppressed. So it does no longer show a scary warning in red, but still acknowledges the overflow happening. --- *By submitting this PR, you agree to release your contribution under the project's CC-BY-NC-ND license.* --- <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-04-16 01:58:14 -05:00

GiteaMirror closed this issue

2026-04-16 01:58:16 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/cs249r_book#3789