ageron/handson-mlp

[bug] The final shape after a tensor [2, 32, 70, 120] is going through the depthwise max pooling is incorrect

Open

#40 opened on May 23, 2026

View on GitHub
 (0 comments) (0 reactions) (1 assignee)Jupyter Notebook (513 forks)github user discovery
bughelp wanted

Repository metrics

Stars
 (1,561 stars)
PR merge metrics
 (PR metrics pending)

Description

Enter the chapter number

Chapter 12 Deep Computer Vision using CNNs

Enter the page number

Page 433

What is the cell's number in the notebook

Line 11 or the last bullet point

Enter the environment you are using to run the notebook

None

Describe your issue

The final shape after a tensor [2, 32, 70, 120] is going through the depthwise max pooling is not [2,8,50,100].

Enter what you expected to happen

The final tensor should be [2,8,70,120], not [2,8,50,100]

If you found a workaround, describe it here

The final tensor should be [2,8,70,120]

Contributor guide

Research direction
Check the notebook cell for depthwise max pooling implementation and verify the kernel size, stride, and padding parameters to ensure the output shape is [2,8,70,120] as expected.
Tech stack
pythonpytorch
Domain
ai
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
PythonPyTorch
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70